Thanks Colin. --Send from my Sony mobile. On Apr 13, 2013 4:06 AM, "Colin McCabe" <cmcc...@alumni.cmu.edu> wrote:
> Hi Azuryy, > > The branch adds new APT documentation which describes the new configuration > that is needed. > It's > in > ./hadoop-hdfs-project/hadoop-hdfs/src/site/apt/ShortCircuitLocalReads.apt.vm > > best, > Colin > > > On Thu, Apr 11, 2013 at 6:37 PM, Azuryy Yu <azury...@gmail.com> wrote: > > > It's good to know HDFS-347 win the votes finally. > > > > Does there need some additional configuration to enable these features? > > > > > > > > On Fri, Apr 12, 2013 at 2:05 AM, Colin McCabe <cmcc...@alumni.cmu.edu > > >wrote: > > > > > 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 > > > > > > > > > >
