The merge vote is now closed. With three +1s, it passes. thanks, Colin
On Wed, Apr 10, 2013 at 10:00 PM, Aaron T. Myers <a...@cloudera.com> wrote: > I'm +1 as well. I've reviewed much of the code as well and have personally > seen it running in production at several different sites. I agree with Todd > that it's a substantial improvement in operability. > > Best, > Aaron > > On Apr 8, 2013, at 1:19 PM, Todd Lipcon <t...@cloudera.com> wrote: > > > +1 for the branch merge. I've reviewed all of the code in the branch, and > > we have people now running this code in production scenarios. It is as > > functional as the old version and way easier to set up/configure. > > > > -Todd > > > > On Mon, Apr 1, 2013 at 4:32 PM, Colin McCabe <cmcc...@alumni.cmu.edu> > wrote: > > > >> Hi all, > >> > >> I think it's time to merge the HDFS-347 branch back to trunk. It's been > >> under > >> review and testing for several months, and provides both a performance > >> advantage, and the ability to use short-circuit local reads without > >> compromising system security. > >> > >> Previously, we tried to merge this and the objection was brought up > that we > >> should keep the old, insecure short-circuit local reads around so that > >> platforms for which secure SCR had not yet been implemented could use it > >> (e.g. Windows). This has been addressed-- see HDFS-4538 for details. > >> Suresh has also volunteered to maintain the insecure SCR code until > secure > >> SCR can be implemented for Windows. > >> > >> Please cast your vote by EOD Monday 4/8. > >> > >> best, > >> Colin > >> > > > > > > > > -- > > Todd Lipcon > > Software Engineer, Cloudera >
