Re: Review Request 34148: Enhancing the StateManager.changeState result.

2015-05-13 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34148/#review83623 --- Ship it!

Re: Review Request 34148: Enhancing the StateManager.changeState result.

2015-05-13 Thread Bill Farner
On May 13, 2015, 1:53 a.m., Aurora ReviewBot wrote: Master (bf7f9b7) is red with this patch. ./build-support/jenkins/build.sh :distZip :assemble :compileJmhJavaNote:

Re: Review Request 34148: Enhancing the StateManager.changeState result.

2015-05-13 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34148/ --- (Updated May 13, 2015, 5:50 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 34148: Enhancing the StateManager.changeState result.

2015-05-13 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34148/#review83643 --- Ship it! Master (7eba711) is green with this patch.

Re: Review Request 34148: Enhancing the StateManager.changeState result.

2015-05-13 Thread Maxim Khutornenko
On May 13, 2015, 4:36 p.m., Bill Farner wrote: src/main/java/org/apache/aurora/scheduler/state/StateChangeResult.java, line 17 https://reviews.apache.org/r/34148/diff/1/?file=957881#file957881line17 s/Describes t/T/ Done. On May 13, 2015, 4:36 p.m., Bill Farner wrote:

Re: Review Request 34148: Enhancing the StateManager.changeState result.

2015-05-12 Thread Maxim Khutornenko
On May 13, 2015, 1:53 a.m., Aurora ReviewBot wrote: Master (bf7f9b7) is red with this patch. ./build-support/jenkins/build.sh :distZip :assemble :compileJmhJavaNote: