Re: Review Request 32181: Only warn about insufficient GC resources when actually needed

2015-03-17 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32181/#review76839 --- Ship it! Master (a419863) is green with this patch.

Re: Review Request 32181: Only warn about insufficient GC resources when actually needed

2015-03-17 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32181/ --- (Updated March 17, 2015, 11:50 p.m.) Review request for Aurora. Summary

Re: Review Request 32181: Only warn about insufficient GC resources when actually needed

2015-03-17 Thread Bill Farner
On March 17, 2015, 11:12 p.m., Maxim Khutornenko wrote: src/main/java/org/apache/aurora/scheduler/async/GcExecutorLauncher.java, line 208 https://reviews.apache.org/r/32181/diff/1/?file=898407#file898407line208 This reordering will result in pulsing the hostname even when an

Re: Review Request 32181: Only warn about insufficient GC resources when actually needed

2015-03-17 Thread Maxim Khutornenko
On March 17, 2015, 11:12 p.m., Maxim Khutornenko wrote: src/main/java/org/apache/aurora/scheduler/async/GcExecutorLauncher.java, line 208 https://reviews.apache.org/r/32181/diff/1/?file=898407#file898407line208 This reordering will result in pulsing the hostname even when an