----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54774/#review159357 -----------------------------------------------------------
Master (8e37d0f) is red with this patch. ./build-support/jenkins/build.sh Test coverage missing for org/apache/aurora/scheduler/reconciliation/TaskTimeout$TimedOutTaskHandler Test coverage missing for org/apache/aurora/scheduler/cron/quartz/AuroraCronJob$CronBatchWorker Test coverage missing for org/apache/aurora/scheduler/cron/quartz/AuroraCronJob Test coverage missing for org/apache/aurora/scheduler/cron/quartz/AuroraCronJob$Config Test coverage missing for org/apache/aurora/scheduler/scheduling/TaskGroups$TaskGroupBatchWorker Test coverage missing for org/apache/aurora/scheduler/scheduling/TaskGroups$1 Test coverage missing for org/apache/aurora/scheduler/scheduling/TaskGroups Test coverage missing for org/apache/aurora/scheduler/storage/mem/MemTaskStore$SecondaryIndex$2 Test coverage missing for org/apache/aurora/scheduler/storage/mem/MemTaskStore$SecondaryIndex$1 Test coverage missing for org/apache/aurora/scheduler/mesos/TaskStatusStats$3 Test coverage missing for org/apache/aurora/scheduler/mesos/TaskStatusStats$2 Test coverage missing for org/apache/aurora/scheduler/mesos/TaskStatusStats$1 Test coverage missing for org/apache/aurora/scheduler/mesos/TaskStatusStats Test coverage missing for org/apache/aurora/scheduler/thrift/AuditMessages Test coverage missing for org/apache/aurora/scheduler/preemptor/PendingTaskProcessor Test coverage missing for org/apache/aurora/scheduler/preemptor/PendingTaskProcessor$1 Test coverage missing for org/apache/aurora/scheduler/preemptor/PendingTaskProcessor$2 Test coverage missing for org/apache/aurora/scheduler/preemptor/PendingTaskProcessor$3 Test coverage missing for org/apache/aurora/scheduler/preemptor/PreemptorModule$PreemptorService Test coverage missing for org/apache/aurora/scheduler/events/WebhookModule Test coverage missing for org/apache/aurora/scheduler/events/Webhook Test coverage missing for org/apache/aurora/scheduler/events/WebhookInfo Test coverage missing for org/apache/aurora/scheduler/storage/log/EntrySerializer$EntrySerializerImpl$1 Test coverage missing for org/apache/aurora/scheduler/storage/log/LogStorage$Settings Test coverage missing for org/apache/aurora/scheduler/storage/log/LogStorage$ScheduledExecutorSchedulingService Test coverage missing for org/apache/aurora/scheduler/storage/log/LogStorageModule Test coverage missing for org/apache/aurora/scheduler/storage/backup/BackupModule Test coverage missing for org/apache/aurora/scheduler/TaskVars Test coverage missing for org/apache/aurora/scheduler/SchedulerLifecycle$DefaultDelayedActions Test coverage missing for org/apache/aurora/scheduler/TierManager$TierManagerImpl$TierConfig Test coverage missing for org/apache/aurora/scheduler/TaskVars$Counter Test coverage missing for org/apache/aurora/scheduler/TaskVars$1 * Try: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. ============================================================================== BUILD FAILED Total time: 5 mins 24.358 secs I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Dec. 15, 2016, 7:24 p.m., David McLaughlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54774/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2016, 7:24 p.m.) > > > Review request for Aurora, Joshua Cohen, Santhosh Kumar Shanmugham, and > Zameer Manji. > > > Bugs: AURORA-1861 > https://issues.apache.org/jira/browse/AURORA-1861 > > > Repository: aurora > > > Description > ------- > > Motivation: Thanks to the mybatis query metrics we added, we found that > double writing Snapshot fields for H2 stores adds considerable overhead to > our snapshot creation time. > > Snapshots are also written as backups, and many operators choose to process > backups offline for analytics, rather than query the live scheduler (due to > not being able to scale reads horizontally). So this allows operators to > enable/disable the hydrated fields as needed. > > > Diffs > ----- > > RELEASE-NOTES.md 90c4793b218f54ad92ab4da5049d5e92c3a104b3 > > src/main/java/org/apache/aurora/scheduler/storage/backup/TemporaryStorage.java > 3fa408e283f91b313633959ea6d2e730d4dc0771 > src/main/java/org/apache/aurora/scheduler/storage/log/LogStorageModule.java > 7dcd1bfcf303cf374e9a6627cb6c632ccea098f2 > > src/main/java/org/apache/aurora/scheduler/storage/log/SnapshotStoreImpl.java > 853780bc68400489578ed3692920aafcec42c999 > src/test/java/org/apache/aurora/scheduler/storage/backup/RecoveryTest.java > 7a11850e217dcb0148e4a4d33542c95b2e53a726 > > src/test/java/org/apache/aurora/scheduler/storage/log/SnapshotStoreImplIT.java > cf0a8f3ea11e9c48d1f16441af54dc781b33bdfc > > Diff: https://reviews.apache.org/r/54774/diff/ > > > Testing > ------- > > ./gradlew test > > I'll apply this to one of our test clusters before merging to master too. > > > Thanks, > > David McLaughlin > >
