Re: Review Request 25710: Remove the JobUpdateAction.INSTANCE_SKIPPED.

2014-09-16 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25710/#review53585 --- services.js in the commit I shipped today has two references to

Re: Review Request 25710: Remove the JobUpdateAction.INSTANCE_SKIPPED.

2014-09-16 Thread Bill Farner
On Sept. 16, 2014, 8:49 p.m., David McLaughlin wrote: services.js in the commit I shipped today has two references to INSTANCES_SKIPPED. Aha, you saw the writing on the wall and snuck in :-P - Bill --- This is an automatically

Re: Review Request 25710: Remove the JobUpdateAction.INSTANCE_SKIPPED.

2014-09-16 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25710/ --- (Updated Sept. 16, 2014, 8:55 p.m.) Review request for Aurora and David

Re: Review Request 25710: Remove the JobUpdateAction.INSTANCE_SKIPPED.

2014-09-16 Thread David McLaughlin
On Sept. 16, 2014, 8:49 p.m., David McLaughlin wrote: services.js in the commit I shipped today has two references to INSTANCES_SKIPPED. Bill Farner wrote: Aha, you saw the writing on the wall and snuck in :-P :) I'm fine with you just punting on changing them and leaving them for

Re: Review Request 25710: Remove the JobUpdateAction.INSTANCE_SKIPPED.

2014-09-16 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25710/#review53592 --- Ship it! Ship It! - David McLaughlin On Sept. 16, 2014, 8:55