accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Bill Havanki <bhava...@clouderagovt.com>
Subject Re: State of ACCUMULO-378 -- Replication
Date Fri, 30 May 2014 20:35:21 GMT
Thanks for arranging those. I've been socked in with testing work this
week, so I'll look at them starting next week :)


On Wed, May 28, 2014 at 11:07 PM, Josh Elser <josh.elser@gmail.com> wrote:

> On 5/28/14, 10:48 AM, Josh Elser wrote:
>
>> On 5/28/14, 9:52 AM, Keith Turner wrote:
>>
>>> On Wed, May 28, 2014 at 9:01 AM, Bill Havanki
>>> <bhavanki@clouderagovt.com>wrote:
>>>
>>>  Is it possible to break the review into smaller, easier-to-review
>>>> chunks?
>>>>
>>>>
>>> I like that.  Could construct a dependency graph of the chunks to be
>>> reviewed and post that as a map for us.
>>>
>>>
>>>
>> I'll see if I can get something up. As Sean asked in IRC yesterday, I
>> still need to write user facing documentation too. I also need to
>> update/finish my state diagrams that I started way back when, which
>> might help in this process.
>>
>> FWIW, you can already go through the review I have up by breaking it
>> into maven modules (e.g. stuff that runs in the master, tserver,
>> client-side, etc). I don't think there's anything presently that
>> wouldn't make sense across modules (maybe GC and Master interaction --
>> those might get combined into one "review").
>>
>> Also, fair warning, but the breakdown of patches if I try to chunk them
>> won't accurately represent how they will be committed to the repo in the
>> final iteration. I assume that's fine for people.
>>
>
> Oof, ok, hopefully these are kosher...
>
> "Client" https://reviews.apache.org/r/22003/
> Master+GC https://reviews.apache.org/r/22004/
> tserver https://reviews.apache.org/r/22005/
> high-level tests https://reviews.apache.org/r/22006/
>



-- 
// Bill Havanki
// Solutions Architect, Cloudera Govt Solutions
// 443.686.9283

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message