Re: Review Request 34121: Disable state transition logging for unknown tasks.

2015-05-12 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34121/#review83484 --- Ship it! Master (759ef68) is green with this patch.

Re: Review Request 34121: Disable state transition logging for unknown tasks.

2015-05-12 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34121/#review83477 --- @ReviewBot retry - Maxim Khutornenko On May 12, 2015, 8:40 p.m.,

Re: Review Request 34121: Disable state transition logging for unknown tasks.

2015-05-12 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34121/#review83481 --- What's the motivation here? Seems like attempted state transitions

Re: Review Request 34121: Disable state transition logging for unknown tasks.

2015-05-12 Thread Maxim Khutornenko
On May 12, 2015, 9:22 p.m., Bill Farner wrote: What's the motivation here? Seems like attempted state transitions for unknown tasks should be transient and/or represent a bug. In those cases, i would assume we definitely want logging. Is there somthing i'm overlooking? The idea