-
Re: code review: HBASE-4608 HLog compressionTodd Lipcon 2012-03-12, 17:58
Hey Ted,
Any chance you've been able to run YCSB on a small cluster for some
performance results? Since this is a perf improvement, it would be
great to have numbers. (I don't think it's a pre-req for commit, but
definitely nice-to-have)
-Todd
On Mon, Mar 12, 2012 at 10:16 AM, Ted Yu <[EMAIL PROTECTED]> wrote:
> The latest patch version is 22.
>
> Please take a look.
>
> On Tue, Mar 6, 2012 at 5:22 PM, Ted Yu <[EMAIL PROTECTED]> wrote:
>
>> Patch v19, the latest patch which addressed Todd's review, passed unit
>> tests.
>>
>> FYI
>>
>>
>> On Mon, Mar 5, 2012 at 3:50 PM, Ted Yu <[EMAIL PROTECTED]> wrote:
>>
>>> Hi,
>>> HBASE-4608 has gone through several iterations of code reviews.
>>> Lars has marked 0.94 as release for HBASE-4608 HLog compression
>>>
>>> Please provide your review comments on https://reviews.apache.org/r/4185/
>>>
>>> Thanks
>>>
>>
>>
--
Todd Lipcon
Software Engineer, Cloudera
Any chance you've been able to run YCSB on a small cluster for some
performance results? Since this is a perf improvement, it would be
great to have numbers. (I don't think it's a pre-req for commit, but
definitely nice-to-have)
-Todd
On Mon, Mar 12, 2012 at 10:16 AM, Ted Yu <[EMAIL PROTECTED]> wrote:
> The latest patch version is 22.
>
> Please take a look.
>
> On Tue, Mar 6, 2012 at 5:22 PM, Ted Yu <[EMAIL PROTECTED]> wrote:
>
>> Patch v19, the latest patch which addressed Todd's review, passed unit
>> tests.
>>
>> FYI
>>
>>
>> On Mon, Mar 5, 2012 at 3:50 PM, Ted Yu <[EMAIL PROTECTED]> wrote:
>>
>>> Hi,
>>> HBASE-4608 has gone through several iterations of code reviews.
>>> Lars has marked 0.94 as release for HBASE-4608 HLog compression
>>>
>>> Please provide your review comments on https://reviews.apache.org/r/4185/
>>>
>>> Thanks
>>>
>>
>>
--
Todd Lipcon
Software Engineer, Cloudera
