Re: Review Request 32359: Adding a configurable delay into writing a backup file.

2015-03-23 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32359/#review77471 --- Ship it! - Stephan Erb On March 23, 2015, 7:10 p.m., Maxim

Re: Review Request 32359: Adding a configurable delay into writing a backup file.

2015-03-23 Thread Maxim Khutornenko
On March 23, 2015, 6:22 p.m., Aurora ReviewBot wrote: Master (a3a35e9) is red with this patch. ./build-support/jenkins/build.sh src.test.python.apache.aurora.client.factory . SUCCESS

Re: Review Request 32359: Adding a configurable delay into writing a backup file.

2015-03-23 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32359/ --- (Updated March 23, 2015, 6:10 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 32359: Adding a configurable delay into writing a backup file.

2015-03-23 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32359/#review77436 --- Master (a3a35e9) is red with this patch.

Re: Review Request 32359: Adding a configurable delay into writing a backup file.

2015-03-23 Thread Maxim Khutornenko
On March 21, 2015, 12:14 p.m., Stephan Erb wrote: src/main/java/org/apache/aurora/scheduler/storage/backup/BackupModule.java, line 66 https://reviews.apache.org/r/32359/diff/1/?file=902147#file902147line66 The description should state the intent of the option more explicitly.

Re: Review Request 32359: Adding a configurable delay into writing a backup file.

2015-03-21 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32359/#review77337 ---

Review Request 32359: Adding a configurable delay into writing a backup file.

2015-03-20 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32359/ --- Review request for Aurora and Bill Farner. Bugs: AURORA-1211

Re: Review Request 32359: Adding a configurable delay into writing a backup file.

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