Re: Review Request 36561: Use the application-level storage lock to prevent DB-level deadlock when GCing rows.

2015-07-16 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36561/#review91995 --- Ship it! Ship It! - Kevin Sweeney On July 16, 2015, 4:59 p.m.,

Re: Review Request 36561: Use the application-level storage lock to prevent DB-level deadlock when GCing rows.

2015-07-16 Thread Bill Farner
On July 16, 2015, 11:48 p.m., Kevin Sweeney wrote: src/main/java/org/apache/aurora/scheduler/storage/db/RowGarbageCollector.java, line 94 https://reviews.apache.org/r/36561/diff/1/?file=1013918#file1013918line94 `deletedCount.get()` Or just move `deletedCount` and

Re: Review Request 36561: Use the application-level storage lock to prevent DB-level deadlock when GCing rows.

2015-07-16 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36561/ --- (Updated July 16, 2015, 11:59 p.m.) Review request for Aurora and Kevin

Re: Review Request 36561: Use the application-level storage lock to prevent DB-level deadlock when GCing rows.

2015-07-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36561/#review91996 --- Ship it! Master (a654b28) is green with this patch.