Re: Review Request 32106: Changed the updater to not update an instance if only the job owner changes

2015-03-16 Thread Bill Farner
> On March 16, 2015, 4:30 p.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/updater/InstanceUpdater.java, > > line 159 > > > > > > There is actually a better place to do this type of action

Re: Review Request 32106: Changed the updater to not update an instance if only the job owner changes

2015-03-16 Thread Bill Farner
> On March 16, 2015, 4:30 p.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/updater/InstanceUpdater.java, > > line 159 > > > > > > There is actually a better place to do this type of action

Re: Review Request 32106: Changed the updater to not update an instance if only the job owner changes

2015-03-16 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32106/#review76568 --- src/main/java/org/apache/aurora/scheduler/updater/InstanceUpdater.j

Re: Review Request 32106: Changed the updater to not update an instance if only the job owner changes

2015-03-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32106/#review76560 --- Ship it! Master (e11ed5b) is green with this patch. ./build-suppo