Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 Search Hadoop and all its subprojects:

Switch to Threaded View
HDFS >> mail # dev >> VOTE: HDFS-347 merge


Copy link to this message
-
Re: VOTE: HDFS-347 merge
+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 <[EMAIL PROTECTED]>wrote:

> On Mon, Apr 1, 2013 at 6:58 PM, Colin McCabe <[EMAIL PROTECTED]>
> wrote:
>
> > On Mon, Apr 1, 2013 at 5:04 PM, Suresh Srinivas <[EMAIL PROTECTED]
> >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 <[EMAIL PROTECTED]>
> >> 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/
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB