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
HBase >> mail # dev >> Someone to look at HBASE-9543?


Copy link to this message
-
Re: Someone to look at HBASE-9543?
There is not any special reason.

Just because we are used to use phabricator to review code.

I will re-post the review to reviews.apache.org if needed.

-liushaohui

On 10/21/2013 10:56 PM, Jean-Marc Spaggiari wrote:
> You're right. I did not figured that. Did not even clicked on the link ;)
>
> I don't think there was any particular reason.
>
> Thanks for looking at it.
>
> JM
>
>
> 2013/10/21 Ted Yu <[EMAIL PROTECTED]>
>
>> Will take a look when I get to office.
>>
>> One question: was there particular reason for review request to be posted
>> on reviews.facebook.net ?
>> If I am not mistaken, this is for Apache trunk, right ?
>>
>> Most people post on reviews.apache.org
>>
>> Cheers
>>
>>
>> On Mon, Oct 21, 2013 at 6:50 AM, Jean-Marc Spaggiari <
>> [EMAIL PROTECTED]> wrote:
>>
>>> Hi,
>>>
>>> One of our users proposed a patch under
>>> HBASE-9543<https://issues.apache.org/jira/browse/HBASE-9543>
>>> .
>>>
>>> I exchanged a bit with him and patch seems to be good now.
>>>
>>> Can a commiter have a look and provide comments? 2 other +1 are welcome.
>>>
>>> Thanks,
>>>
>>> JM
>>>
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