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
Hi Colin,

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.  Thanks a lot!

BTW, you should conclude the VOTE you started and use a separated thread for other discussion as Suresh mentioned earlier.  I hope you could use our convention on voting.  Otherwise, it is hard for others to follow.

Tsz-Wo
________________________________
 From: Suresh Srinivas <[EMAIL PROTECTED]>
To: "[EMAIL PROTECTED]" <[EMAIL PROTECTED]>
Sent: Wednesday, March 6, 2013 5:09 AM
Subject: Re: VOTE: HDFS-347 merge
 
Thanks Colin. Will check it out as soon as I can.
On Tue, Mar 5, 2013 at 12:24 PM, Colin McCabe <[EMAIL PROTECTED]>wrote:

> On Tue, Feb 26, 2013 at 5:09 PM, Suresh Srinivas <[EMAIL PROTECTED]>
> wrote:
> >>
> >> Suresh, if you're willing to "support and maintain" HDFS-2246, do you
> >> have cycles to propose a patch to the HDFS-347 branch reintegrating
> >> HDFS-2246 with the simplifications you outlined? In your review, did
> >> you find anything else you'd like to address prior to the merge, or is
> >> this the only item? -C
> >
> >
> > Yes, I can work on adding HDFS-2246 back in HDFS-347 branch. I will get
> to
> > it
> > in a week or two, if that is okay.
> >
> > I have not looked at the patch closely. But I think any issues found
> could
> > be fixed in trunk and should not block the merge.
>
> Hi Suresh,
>
> HDFS-4538 adds the ability to use the old block reader in the HDFS-347
> branch.  Check it out.
>
> Colin
>

--
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