+1 for the merge. Some of the minor changes or discussions happening in some of the related tasks can be addressed post merge to trunk. The pending discussions/issues should be fixed before this feature gets merged into branch-2.
On Tue, Apr 2, 2013 at 10:38 AM, Colin McCabe <cmcc...@alumni.cmu.edu>wrote: > On Mon, Apr 1, 2013 at 6:58 PM, Colin McCabe <cmcc...@alumni.cmu.edu> > wrote: > > > On Mon, Apr 1, 2013 at 5:04 PM, Suresh Srinivas <sur...@hortonworks.com > >wrote: > > > >> Colin, > >> > >> For the record, the last email in the previous thread in ended with the > >> following comment from Nicholas: > >> > It is great to hear that you agree to keep HDFS-2246. Please as well > >> address my comments posted on HDFS-347 and let me know once you have > >> posted > >> a new patch on HDFS-347. > >> > >> > > Hi Nicholas, > > > > Can you please open a JIRA listing what you think should be fixed or > > changed, and why? > > > > Also please specify whether it is important to fix this before the merge, > > and if so, why. If this is a minor style change, or renaming function X > to > > Y, then I think we can easily do it after the merge. > > > > thanks, > > Colin > > > > > Hi Nicholas, > > I opened https://issues.apache.org/jira/browse/HDFS-4661 with some of the > style fixes that you suggested in HDFS-347. If there is anything else you > would like to see addressed before the merge, please add it to this JIRA. > > thanks, > Colin > > > > > > > > >> I did not see any response (unless I missed it). Can you please address > >> it? > >> > >> Regards, > >> Suresh > >> > >> > >> 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 > >> > > >> > >> > >> > >> -- > >> http://hortonworks.com/download/ > >> > > > > > -- http://hortonworks.com/download/