Re: Review Request 33869: Fix inconsistency in MemTaskStore secondary indices.

2015-05-06 Thread Bill Farner
> On May 6, 2015, 3:39 p.m., Maxim Khutornenko wrote: > > Mind explaining explaining the bug for everyone's benefit? > > Maxim Khutornenko wrote: > Never mind, found the answer in the attached ticket. Hard to notice > "index.remove(key, Tasks.id(task));" bundled with unrelated stats refacto

Re: Review Request 33869: Fix inconsistency in MemTaskStore secondary indices.

2015-05-06 Thread Maxim Khutornenko
> On May 6, 2015, 3:39 p.m., Maxim Khutornenko wrote: > > Mind explaining explaining the bug for everyone's benefit? Never mind, found the answer in the attached ticket. Hard to notice "index.remove(key, Tasks.id(task));" bundled with unrelated stats refactoring. - Maxim ---

Re: Review Request 33869: Fix inconsistency in MemTaskStore secondary indices.

2015-05-06 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33869/#review82690 --- Ship it! Ship It! - Maxim Khutornenko On May 5, 2015, 11:16 p.m.

Re: Review Request 33869: Fix inconsistency in MemTaskStore secondary indices.

2015-05-06 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33869/#review82683 --- Mind explaining explaining the bug for everyone's benefit? - Maxim

Re: Review Request 33869: Fix inconsistency in MemTaskStore secondary indices.

2015-05-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33869/#review82614 --- Ship it! Master (751d65f) is green with this patch. ./build-suppo

Review Request 33869: Fix inconsistency in MemTaskStore secondary indices.

2015-05-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33869/ --- Review request for Aurora and Maxim Khutornenko. Bugs: AURORA-1305 https://