Re: Review Request 42985: Add a flag to configure H2 LOCK_TIMEOUT.

2016-02-01 Thread Zameer Manji
> On Jan. 29, 2016, 3:32 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/storage/db/DbModule.java, line 87 > > > > > > I suggest you raise the default to a very conservative level (like 1 > >

Re: Review Request 42985: Add a flag to configure H2 LOCK_TIMEOUT.

2016-02-01 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42985/ --- (Updated Feb. 1, 2016, 2:44 p.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 42985: Add a flag to configure H2 LOCK_TIMEOUT.

2016-02-01 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42985/#review117227 --- Ship it! Ship It! - Joshua Cohen On Jan. 29, 2016, 11:23

Re: Review Request 42985: Add a flag to configure H2 LOCK_TIMEOUT.

2016-01-29 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42985/#review117065 --- Master (0eb753d) is green with this patch.

Re: Review Request 42985: Add a flag to configure H2 LOCK_TIMEOUT.

2016-01-29 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42985/#review117045 --- Ship it! Ship It! - John Sirois On Jan. 29, 2016, 4:23

Re: Review Request 42985: Add a flag to configure H2 LOCK_TIMEOUT.

2016-01-29 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42985/#review117049 --- Fix it, then Ship it!