Re: Review Request 63739: OOZIE-3113 Retry for ZK lock release

2018-01-02 Thread András Piros via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63739/#review194623 --- Ship it! Ship It! - András Piros On Dec. 27, 2017, 10:13 p.m

Re: Review Request 63739: OOZIE-3113 Retry for ZK lock release

2017-12-27 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63739/#review194554 --- Ship it! Please commit after you get +1 from András as well -

Re: Review Request 63739: OOZIE-3113 Retry for ZK lock release

2017-12-27 Thread Satish Saley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63739/ --- (Updated Dec. 27, 2017, 2:13 p.m.) Review request for oozie. Bugs: OOZIE-3113

Re: Review Request 63739: OOZIE-3113 Retry for ZK lock release

2017-12-27 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63739/#review194551 --- core/src/test/java/org/apache/oozie/service/TestZKLocksService.ja

Re: Review Request 63739: OOZIE-3113 Retry for ZK lock release

2017-12-22 Thread Satish Saley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63739/ --- (Updated Dec. 22, 2017, 4:43 p.m.) Review request for oozie. Bugs: OOZIE-3113

Re: Review Request 63739: OOZIE-3113 Retry for ZK lock release

2017-12-22 Thread Satish Saley
> On Nov. 30, 2017, 9:33 a.m., András Piros wrote: > > core/src/main/java/org/apache/oozie/service/ZKLocksService.java > > Lines 239 (patched) > > > > > > Yep, blocking a thread for 30-ish minutes doesn't seem like a

Re: Review Request 63739: OOZIE-3113 Retry for ZK lock release

2017-12-20 Thread Rohini Palaniswamy
> On Nov. 30, 2017, 5:33 p.m., András Piros wrote: > > core/src/main/java/org/apache/oozie/service/ZKLocksService.java > > Lines 239 (patched) > > > > > > Yep, blocking a thread for 30-ish minutes doesn't seem like a

Re: Review Request 63739: OOZIE-3113 Retry for ZK lock release

2017-12-20 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63739/#review194265 --- core/src/main/java/org/apache/oozie/service/ZKLocksService.java L

Re: Review Request 63739: OOZIE-3113 Retry for ZK lock release

2017-11-30 Thread András Piros via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63739/#review192309 --- core/src/main/java/org/apache/oozie/service/ZKLocksService.java L

Re: Review Request 63739: OOZIE-3113 Retry for ZK lock release

2017-11-27 Thread Satish Saley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63739/ --- (Updated Nov. 27, 2017, 11:22 a.m.) Review request for oozie. Bugs: OOZIE-311

Review Request 63739: OOZIE-3113 Retry for ZK lock release

2017-11-10 Thread Satish Saley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63739/ --- Review request for oozie. Bugs: OOZIE-3113 https://issues.apache.org/jira/b