Re: Review Request 24827: Add a pubsub event adapter to the JobUpdateController.

2014-08-19 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24827/#review50976 ---

Re: Review Request 24827: Add a pubsub event adapter to the JobUpdateController.

2014-08-19 Thread Bill Farner
On Aug. 19, 2014, 3:58 p.m., Maxim Khutornenko wrote: src/main/java/org/apache/aurora/scheduler/updater/JobUpdateController.java, line 74 https://reviews.apache.org/r/24827/diff/1/?file=663946#file663946line74 A TODO to follow up on deleted flag removal? Removed now, thanks for

Re: Review Request 24827: Add a pubsub event adapter to the JobUpdateController.

2014-08-19 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24827/#review50995 --- Ship it! Ship It! - Maxim Khutornenko On Aug. 19, 2014, 5:43

Re: Review Request 24827: Add a pubsub event adapter to the JobUpdateController.

2014-08-19 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24827/ --- (Updated Aug. 19, 2014, 5:49 p.m.) Review request for Aurora and Maxim