Re: Review Request 30433: Prevent multiple active job updates from being inserted into storage.

2015-01-30 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30433/#review70418 --- Ship it! Master (4f04a34) is green with this patch. ./build-suppo

Re: Review Request 30433: Prevent multiple active job updates from being inserted into storage.

2015-01-30 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30433/ --- (Updated Jan. 30, 2015, 6:34 p.m.) Review request for Aurora and Maxim Khutorne

Re: Review Request 30433: Prevent multiple active job updates from being inserted into storage.

2015-01-30 Thread Bill Farner
> On Jan. 30, 2015, 5:44 a.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/updater/JobUpdateControllerImpl.java, > > line 152 > > > > > > Suggest adding an explanation of what active update

Re: Review Request 30433: Prevent multiple active job updates from being inserted into storage.

2015-01-29 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30433/#review70316 --- Ship it! src/main/java/org/apache/aurora/scheduler/updater/JobUpda

Re: Review Request 30433: Prevent multiple active job updates from being inserted into storage.

2015-01-29 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30433/#review70302 --- Ship it! Master (4f04a34) is green with this patch. ./build-suppo

Review Request 30433: Prevent multiple active job updates from being inserted into storage.

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