Review Request 25300: Add a state machine to react to job update status changes.

2014-09-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25300/ --- Review request for Aurora, Joshua Cohen and Maxim Khutornenko. Bugs:

Re: Review Request 25300: Add a state machine to react to job update status changes.

2014-09-03 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25300/#review52193 --- Ship it! Ship It! - Joshua Cohen On Sept. 3, 2014, 4:02 p.m.,

Re: Review Request 25300: Add a state machine to react to job update status changes.

2014-09-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25300/#review52195 ---

Re: Review Request 25300: Add a state machine to react to job update status changes.

2014-09-03 Thread Bill Farner
On Sept. 3, 2014, 5:33 p.m., Maxim Khutornenko wrote: src/main/java/org/apache/aurora/scheduler/updater/JobUpdateStateMachine.java, lines 74-76 https://reviews.apache.org/r/25300/diff/1/?file=675222#file675222line74 This will also throw for in-place transitions (e.g.

Re: Review Request 25300: Add a state machine to react to job update status changes.

2014-09-03 Thread Maxim Khutornenko
On Sept. 3, 2014, 5:33 p.m., Maxim Khutornenko wrote: src/main/java/org/apache/aurora/scheduler/updater/JobUpdateStateMachine.java, lines 74-76 https://reviews.apache.org/r/25300/diff/1/?file=675222#file675222line74 This will also throw for in-place transitions (e.g.

Re: Review Request 25300: Add a state machine to react to job update status changes.

2014-09-03 Thread Bill Farner
On Sept. 3, 2014, 5:33 p.m., Maxim Khutornenko wrote: src/main/java/org/apache/aurora/scheduler/updater/JobUpdateStateMachine.java, lines 74-76 https://reviews.apache.org/r/25300/diff/1/?file=675222#file675222line74 This will also throw for in-place transitions (e.g.

Re: Review Request 25300: Add a state machine to react to job update status changes.

2014-09-03 Thread Bill Farner
On Sept. 3, 2014, 5:33 p.m., Maxim Khutornenko wrote: src/main/java/org/apache/aurora/scheduler/updater/JobUpdateStateMachine.java, lines 74-76 https://reviews.apache.org/r/25300/diff/1/?file=675222#file675222line74 This will also throw for in-place transitions (e.g.

Re: Review Request 25300: Add a state machine to react to job update status changes.

2014-09-03 Thread Maxim Khutornenko
On Sept. 3, 2014, 5:33 p.m., Maxim Khutornenko wrote: src/main/java/org/apache/aurora/scheduler/updater/JobUpdateStateMachine.java, lines 74-76 https://reviews.apache.org/r/25300/diff/1/?file=675222#file675222line74 This will also throw for in-place transitions (e.g.

Re: Review Request 25300: Add a state machine to react to job update status changes.

2014-09-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25300/#review52231 --- Ship it! Ship It! - Maxim Khutornenko On Sept. 3, 2014, 4:02