----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32359/#review77337 -----------------------------------------------------------
src/main/java/org/apache/aurora/scheduler/storage/backup/BackupModule.java <https://reviews.apache.org/r/32359/#comment125247> The description should state the intent of the option more explicitly. Even reading the code did not help me much. Only the description in the associated Jira ticket helps to undestand why this option is important and might need to be tuned. - Stephan Erb On March 21, 2015, 3:22 a.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32359/ > ----------------------------------------------------------- > > (Updated March 21, 2015, 3:22 a.m.) > > > Review request for Aurora and Bill Farner. > > > Bugs: AURORA-1211 > https://issues.apache.org/jira/browse/AURORA-1211 > > > Repository: aurora > > > Description > ------- > > Adding a configurable delay into writing a backup file. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/storage/backup/BackupModule.java > a6b187d888726b921733a36fcaecdab97bdef094 > src/main/java/org/apache/aurora/scheduler/storage/backup/StorageBackup.java > 569648aea2ccdb57663d16b7c837fd2677694419 > src/test/java/org/apache/aurora/scheduler/storage/backup/RecoveryTest.java > 5853c037d53e707e5df434fc661cd285ed9ecfc4 > > src/test/java/org/apache/aurora/scheduler/storage/backup/StorageBackupTest.java > ebe551739fb6b7132ce666ad9b3c5a86e90a5363 > > Diff: https://reviews.apache.org/r/32359/diff/ > > > Testing > ------- > > ./gradlew -Pq build > > > Thanks, > > Maxim Khutornenko > >