Mailing-List: contact hdfs-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: hdfs-dev@hadoop.apache.org
Received-SPF: neutral (athena.apache.org: local policy)
Content-Type: text/plain; charset=iso-8859-1
Mime-Version: 1.0 (Apple Message framework v1283)
Subject: Re: [VOTE] Merge HDFS-3077 (QuorumJournalManager) branch to trunk
From: sanjay Radia <sanjay@hortonworks.com>
In-Reply-To: 
 <CADY20s4ExvZmM0C4RRaNxg=zT1heAKp6G6Xz0D9erL0BdGXoaw@mail.gmail.com>
Date: Mon, 1 Oct 2012 10:55:41 -0700
Content-Transfer-Encoding: quoted-printable
Message-Id: <93A85775-50B1-45F3-8EC5-C8FBF5D138EA@hortonworks.com>
References: 
 <CADY20s544EPrgFoY8aF15nDHbupSW_rvxt8SWVdRA6KmYogxRg@mail.gmail.com>
 <59E6DFA2-2F24-4B30-84FB-5DD2CCE585E5@hortonworks.com>
 <CADY20s4ExvZmM0C4RRaNxg=zT1heAKp6G6Xz0D9erL0BdGXoaw@mail.gmail.com>
To: hdfs-dev@hadoop.apache.org

Todd,=20
  Even though this work was under development over a period of time,  =
during its development it was not clear when the design was fairly =
stable to begin a thorough review. Hence the time of merge is when the =
real review happens in such large projects.

I have already indicated on the jira that i do not have any =
philosophical objection to this work being in HDFS - hence this should =
not be a worry on your part.=20

The extra week will result in a more through review (hopefully this will =
have a side effect of perhaps easing Konstanine's concern about
HDFS adding such complex code).=20

Lets plan to do the merge next monday.

thanks

sanjay


On Sep 28, 2012, at 3:02 PM, Todd Lipcon wrote:

> Hey Sanjay,
>=20
> While I understand it's substantial and complex code, the code and the
> design doc have been available for several months, and the community
> has certainly been aware of its development. I also gave a heads up
> last week that I would call a merge this week. So I feel like there
> has been sufficient time for interested parties to review.
>=20
> That said, since I was sick for much of this week and not immediately
> responsive to some of the questions from you and Suresh, I'm happy to
> agree to postpone the merge to early next week. Let's extend the vote
> to last until Monday end of day PST.
>=20
> Of course if there are follow-up questions or bugs found after the
> merge, you've all got my phone number and I'm not going anywhere! ;-)
>=20
> Thanks
> -Todd
>=20
> On Fri, Sep 28, 2012 at 12:06 PM, sanjay Radia =
<sanjay@hortonworks.com> wrote:
>> Suresh and I are still reviewing this design and patch.
>> The 3077 code along with  the code pulled from 3092 is fairly =
substrantial. The design is also fairly complex and involved.
>> I  would request that we postpone the merge for another week to give =
folks time to review this fully.
>>=20
>>=20
>> sanjay
>>=20
>>=20
>>=20
>>=20
>>=20
>> On Sep 25, 2012, at 4:02 PM, Todd Lipcon wrote:
>>=20
>>> Dear fellow HDFS developers,
>>>=20
>>> Per my email thread last week ("Heads up: merge for QJM branch soon"
>>> at http://markmail.org/message/vkyh5culdsuxdb6t) I would like to
>>> propose merging the HDFS-3077 branch into trunk. The branch has been
>>> active since mid July and has stabilized significantly over the last
>>> two months. It has passed the full test suite, findbugs, and release
>>> audit, and I think it's ready to merge at this point.
>>>=20
>>> The branch has been fully developed using the standard
>>> 'review-then-commit' (RTC) policy, and the design is described in
>>> detail in a document attached to HDFS-3077 itself. The code itself =
has
>>> been contributed by me, Aaron, and Eli, but I'd be remiss not to =
also
>>> acknowledge the contributions to the design from discussions with
>>> Suresh, Sanjay, Henry Robinson, Patrick Hunt, Ivan Kelly, Andrew
>>> Purtell, Flavio Junqueira, Ben Reed, Nicholas, Bikas, Brandon, and
>>> others. Additionally, special thanks to Andrew Purtell and Stephen =
Chu
>>> for their help with cluster testing.
>>>=20
>>> This initial VOTE is to merge only into trunk, but, following the
>>> pattern of automatic failover, I expect to merge it into branch-2
>>> within a few weeks as well. The merge to branch-2 should be clean, =
as
>>> both I and Andrew Purtell have been testing on branch-2-derived
>>> codebases in addition to trunk.
>>>=20
>>> Please cast your vote by EOD Friday 9/29. Given that the branch has
>>> only had small changes in the last few weeks, and there was a "heads
>>> up" last week, I trust this should be enough time for committers to
>>> cast their votes. Per our by-laws, we need a minimum of three =
binding
>>> +1 votes from committers.
>>>=20
>>> I will start the voting with my own +1.
>>>=20
>>> Thanks
>>> -Todd
>>> --
>>> Todd Lipcon
>>> Software Engineer, Cloudera
>>=20
>=20
>=20
>=20
> --=20
> Todd Lipcon
> Software Engineer, Cloudera