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
