List:Commits« Previous MessageNext Message »
From:Ingo Strüwing Date:November 6 2009 4:48pm
Subject:Re: bzr commit into mysql-6.0-backup branch (charles.bell:2889)
Bug#44787
View as plain text  
Hi Chuck,

Charles Bell, 06.11.2009 15:55:

> Ingo,
> 
> I forgot to mention. This is the newest patch for this work.
> 
> http://lists.mysql.com/commits/89594

What a mess. I thought, our workflow was like this:

  Implementor               Reviewers

  Prepare patch 1

  Prepare patch 2           Review patch 1

  Prepare patch 3           Review patch 2

  Rework patch 1            Review patch 3

  Rework patch 2            Review new patch 1

  ...

But when you rework a patch immediately after *one* reviewer provides
comments and blame the other one to use an old patch, we're not doing as
well as we could.

When you publish a new list of patches every day, but reviewers need 1.5
to 2 days for a review of all three, we're doing a mess, but we're not
doing as well as we could.

In many cases, where congestion is an issue, the motto is "Go slower to
move faster."

Regards
Ingo
-- 
Ingo Strüwing, Database Group
Sun Microsystems GmbH, Sonnenallee 1, D-85551 Kirchheim-Heimstetten
Geschäftsführer: Thomas Schröder,   Wolfgang Engels,   Wolf Frenkel
Vorsitzender des Aufsichtsrates: Martin Häring   HRB München 161028
Thread
bzr commit into mysql-6.0-backup branch (charles.bell:2889) Bug#44787Chuck Bell2 Nov
  • Re: bzr commit into mysql-6.0-backup branch (charles.bell:2889)Bug#44787Ingo Strüwing5 Nov
    • Re: bzr commit into mysql-6.0-backup branch (charles.bell:2889)Bug#44787Charles Bell6 Nov
      • Re: bzr commit into mysql-6.0-backup branch (charles.bell:2889)Bug#44787Charles Bell6 Nov
        • Re: bzr commit into mysql-6.0-backup branch (charles.bell:2889)Bug#44787Ingo Strüwing6 Nov
          • Re: bzr commit into mysql-6.0-backup branch (charles.bell:2889)Bug#44787Charles Bell6 Nov
            • Re: bzr commit into mysql-6.0-backup branch (charles.bell:2889)Bug#44787Ingo Strüwing7 Nov
      • Re: bzr commit into mysql-6.0-backup branch (charles.bell:2889)Bug#44787Ingo Strüwing6 Nov