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 https://reviews.apache.org/r/32106/diff/1/?file=895820#file895820line159 There is actually a better place to do this type of action [1]. If not

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 ---

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.

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

2015-03-16 Thread Steve Niemitz
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32106/ --- Review request for Aurora and Bill Farner. Bugs: AURORA-1200

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 https://reviews.apache.org/r/32106/diff/1/?file=895820#file895820line159 There is actually a better place to do this type of action [1]. If not