Re: Bug with blockAddCompletions decrement

2018-12-04 Thread Ivan Kelly
> > Not an issue with master as blockAddCompletions has been replaced with > > a simple boolean and failure handling changed to only deal with one > > failure at a time. However, looking at it again, I think I did spot a > > similar bug. will dig in after I send this. > > Great; looking forward. I

Re: Bug with blockAddCompletions decrement

2018-12-03 Thread Venkateswara Rao Jujjuri
On Mon, Dec 3, 2018 at 1:40 AM Ivan Kelly wrote: > > This may not be an issue with the master as we moved to immutable > metadata, > > Not an issue with master as blockAddCompletions has been replaced with > a simple boolean and failure handling changed to only deal with one > failure at a time.

Re: Bug with blockAddCompletions decrement

2018-12-03 Thread Ivan Kelly
> This may not be an issue with the master as we moved to immutable metadata, Not an issue with master as blockAddCompletions has been replaced with a simple boolean and failure handling changed to only deal with one failure at a time. However, looking at it again, I think I did spot a similar bug

Bug with blockAddCompletions decrement

2018-12-02 Thread Venkateswara Rao Jujjuri
This may not be an issue with the master as we moved to immutable metadata, but older releases have an issue with not decrementing blockAddCompletions. Sijie/Ivan/Sam can you validate this? blockAddCompletions gets incremented while initiating the ensemble change in handleBookieFailure(). Base