----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33612/#review83525 -----------------------------------------------------------
Master (be75c36) is red with this patch. ./build-support/jenkins/build.sh :distZip :assemble :compileJmhJavaNote: /home/jenkins/jenkins-slave/workspace/AuroraBot/src/jmh/java/org/apache/aurora/benchmark/fakes/FakeSchedulerDriver.java uses or overrides a deprecated API. Note: Recompile with -Xlint:deprecation for details. :processJmhResources UP-TO-DATE :jmhClasses :checkstyleJmh :jsHint :checkstyleMain :compileTestJava/home/jenkins/jenkins-slave/workspace/AuroraBot/src/test/java/org/apache/aurora/scheduler/SchedulerLifecycleTest.java:81: error: method addAction in interface ShutdownRegistry cannot be applied to given types; shutdownRegistry.addAction(capture(shutdownCommand)); ^ required: T#1 found: ExceptionalCommand<CAP#1> reason: inference variable T#2 has incompatible bounds equality constraints: ExceptionalCommand<?> upper bounds: ExceptionalCommand<CAP#2>,T#1,Object where T#1,E,T#2 are type-variables: T#1 extends ExceptionalCommand<E> declared in method <E,T#1>addAction(T#1) E extends Exception declared in method <E,T#1>addAction(T#1) T#2 extends Object declared in method <T#2>capture(Capture<T#2>) where CAP#1,CAP#2 are fresh type-variables: CAP#1 extends Exception from capture of ? CAP#2 extends Exception from capture of ? 1 error FAILED FAILURE: Build failed with an exception. * What went wrong: Execution failed for task ':compileTestJava'. > Compilation failed; see the compiler error output for details. * 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: 2 mins 2.311 secs I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On May 12, 2015, 11:49 p.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33612/ > ----------------------------------------------------------- > > (Updated May 12, 2015, 11:49 p.m.) > > > Review request for Aurora, Joshua Cohen, Kevin Sweeney, and Maxim Khutornenko. > > > Bugs: AURORA-556 > https://issues.apache.org/jira/browse/AURORA-556 > > > Repository: aurora > > > Description > ------- > > Add a task store implementation that uses a relational database. > > The meat of this review is in `DbTaskStore`, `TaskConfigManager`, and the > mapper xml files. Some supporting actors include files under views/, which > serve DB business objects. I suggest reviewers start by skimming > `DbTaskStore` and digesting the comments in there. > > There are some known loose ends here, most notably being continued > performance enhancements and cleanup of relations in different tables. I've > included several relevant TODOs, ~all of which must be addressed before this > becomes the default task store. > > > Diffs > ----- > > api/src/main/thrift/org/apache/aurora/gen/api.thrift > 0182ecb3942cfa2d9dd21138779815f4500339be > examples/vagrant/upstart/aurora-scheduler.conf > cc4864c4d954d58e9bfaaaf5fc5afc8d0e032e78 > src/jmh/java/org/apache/aurora/benchmark/ThriftApiBenchmarks.java > 88d27dd729fd004d1510917a031591addba51816 > src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java > 239c61625bc49e53be918c59056f071b3b6b86b9 > src/main/java/org/apache/aurora/scheduler/storage/db/DBJobUpdateStore.java > ea5600725d5dd84d21ca8d37b560c6d41541d016 > src/main/java/org/apache/aurora/scheduler/storage/db/DbModule.java > 8859ca47088896a1814321147c6b4c31828b3db9 > src/main/java/org/apache/aurora/scheduler/storage/db/DbStorage.java > 1a6c3f21d4fcb476539f588facecd8ef37332837 > src/main/java/org/apache/aurora/scheduler/storage/db/DbTaskStore.java > PRE-CREATION > src/main/java/org/apache/aurora/scheduler/storage/db/DbUtil.java > 7b4067cfe10d8aa1e9e0e8bfbcd0e83c9beb4e7a > src/main/java/org/apache/aurora/scheduler/storage/db/TaskConfigManager.java > PRE-CREATION > src/main/java/org/apache/aurora/scheduler/storage/db/TaskConfigMapper.java > PRE-CREATION > src/main/java/org/apache/aurora/scheduler/storage/db/TaskMapper.java > PRE-CREATION > > src/main/java/org/apache/aurora/scheduler/storage/db/shims/ContainerShim.java > PRE-CREATION > > src/main/java/org/apache/aurora/scheduler/storage/db/shims/TaskConstraintShim.java > PRE-CREATION > > src/main/java/org/apache/aurora/scheduler/storage/db/typehandlers/ScheduleStatusTypeHandler.java > PRE-CREATION > > src/main/java/org/apache/aurora/scheduler/storage/db/typehandlers/TypeHandlers.java > 4d0c10d60037a3310226a6fd8936b84ae4135e7e > > src/main/java/org/apache/aurora/scheduler/storage/db/views/AssignedPort.java > PRE-CREATION > > src/main/java/org/apache/aurora/scheduler/storage/db/views/ScheduledTaskWrapper.java > PRE-CREATION > > src/main/java/org/apache/aurora/scheduler/storage/db/views/TaskConfigRow.java > PRE-CREATION > src/main/java/org/apache/aurora/scheduler/storage/db/views/TaskLink.java > PRE-CREATION > > src/main/java/org/apache/aurora/scheduler/storage/mem/InMemStoresModule.java > 21f7d4db821930d2c5b52648e1996aa1ef12a85c > > src/main/resources/org/apache/aurora/scheduler/storage/db/JobUpdateDetailsMapper.xml > f76f9a988669dab598e9d19f849018c6f55ce03e > > src/main/resources/org/apache/aurora/scheduler/storage/db/TaskConfigMapper.xml > PRE-CREATION > src/main/resources/org/apache/aurora/scheduler/storage/db/TaskMapper.xml > PRE-CREATION > src/test/java/org/apache/aurora/scheduler/state/StateManagerImplTest.java > afb7db8eefa63b84d370877742870acdec58899c > > src/test/java/org/apache/aurora/scheduler/storage/AbstractTaskStoreTest.java > 6a6ff27d8d0f1e537a74c1df981b97e5d8b2f2e8 > > src/test/java/org/apache/aurora/scheduler/storage/db/DbJobUpdateStoreTest.java > 7d856d020da854c125c037f01357e81de93895e1 > src/test/java/org/apache/aurora/scheduler/storage/db/DbTaskStoreTest.java > PRE-CREATION > > src/test/java/org/apache/aurora/scheduler/storage/mem/InMemTaskStoreTest.java > 8f139fc987a98ef0d7f2969720134729411b8b84 > > src/test/java/org/apache/aurora/scheduler/storage/mem/StorageTransactionTest.java > bad9eb56b33c3e649c3b173e83d9c30da8f9317d > > Diff: https://reviews.apache.org/r/33612/diff/ > > > Testing > ------- > > Unit tests and end-to-end tests, both using the new task store by default > with this change. > > > Thanks, > > Bill Farner > >