-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53796/#review155984
-----------------------------------------------------------
Master (e9a182f) is red with this patch.
./build-support/jenkins/build.sh
:commons:jar
:compileJava/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/storage/log/WriteAheadStorage.java:74:
Note: Wrote forwarder
org.apache.aurora.scheduler.storage.log.WriteAheadStorageForwarder
@Forward({
^
Note: Writing
file:/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/classes/main/org/apache/aurora/common/args/apt/cmdline.arg.info.txt.2
Note: Writing
file:/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/classes/main/META-INF/compiler/resource-mappings/org.apache.aurora.common.args.apt.CmdLineProcessor
:generateBuildProperties
:processResources
:classes
:jar
:startScripts
:distTar
: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[ant:checkstyle]
/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/cron/quartz/Quartz.java:21:8:
error: Unused import - com.google.common.base.Throwables.
[ant:checkstyle]
/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/discovery/ServiceDiscoveryModule.java:23:8:
error: Unused import - com.google.common.base.Throwables.
[ant:checkstyle]
/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java:36:8:
error: Unused import - com.google.common.base.Throwables.
[ant:checkstyle]
/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/scheduling/TaskGroups.java:28:8:
error: Unused import - com.google.common.base.Throwables.
FAILED
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':checkstyleMain'.
> Checkstyle rule violations were found. See the report at:
> file:///home/jenkins/jenkins-slave/workspace/AuroraBot/dist/reports/checkstyle/main.html
* 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: 1 mins 13.326 secs
I will refresh this build result if you post a review containing "@ReviewBot
retry"
- Aurora ReviewBot
On Nov. 15, 2016, 10 p.m., Zameer Manji wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/53796/
> -----------------------------------------------------------
>
> (Updated Nov. 15, 2016, 10 p.m.)
>
>
> Review request for Aurora, Joshua Cohen and Stephan Erb.
>
>
> Bugs: AURORA-1821
> https://issues.apache.org/jira/browse/AURORA-1821
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Bump guava dependency to 20.
>
> Release Notes: https://github.com/google/guava/wiki/Release20
>
> It's the usual mix of new features and deprecations. The additions of
> `Quantiles` and `Stats` could give us some quick improvements in our stats
> libraries.
>
>
> Diffs
> -----
>
> build.gradle bbd5e3aea33817e41c9cf2878df99ba6fe44ffec
> src/main/java/org/apache/aurora/scheduler/SchedulerLifecycle.java
> 5ac5f253ca6b3b631474195952b934845dc4df40
> src/main/java/org/apache/aurora/scheduler/TierModule.java
> d9566648d3927dfe57c596c0fbb1210569b8d8d6
> src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java
> 0f92a3c2c39a8117bf30db5991f2781b12e1830f
>
> src/main/java/org/apache/aurora/scheduler/cron/quartz/CronJobManagerImpl.java
> 850effce75bc031eadaf5cc617e6d32e242c3403
> src/main/java/org/apache/aurora/scheduler/cron/quartz/Quartz.java
> c379a32c6d38081c6f5f340c7551aa2a24e315a0
>
> src/main/java/org/apache/aurora/scheduler/discovery/ServiceDiscoveryModule.java
> ef62f2709fdcd0eae2014290fc51441dab7b96c0
> src/main/java/org/apache/aurora/scheduler/events/WebhookModule.java
> 05e19f4a62e2abd75ed88a12ad47a65c57e677f6
> src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java
> 84f7ef3514623fe59c7ef067753fe8a583373a21
> src/main/java/org/apache/aurora/scheduler/http/LeaderRedirect.java
> 0077911b0973483d57a029b80ae7b815f1de3b81
> src/main/java/org/apache/aurora/scheduler/http/api/ApiBeta.java
> 053b87b3e4b7d9995395e372fc11cfd5dc7034c4
>
> src/main/java/org/apache/aurora/scheduler/http/api/GsonMessageBodyHandler.java
> 1007a3263e46141171bb4134d97dc2b1cdfbc883
>
> src/main/java/org/apache/aurora/scheduler/http/api/security/Kerberos5ShiroRealmModule.java
> 0f8bdbbf77dd23c6c370e26a7e1dbc3f1a4ebfe0
>
> src/main/java/org/apache/aurora/scheduler/mesos/CommandLineDriverSettingsModule.java
> 96b5a28afd611e8d07a2e868cc94fd786a8bfc36
> src/main/java/org/apache/aurora/scheduler/scheduling/TaskGroups.java
> 77187bc19fb1f783d1b820d324c7fe18e509365f
> src/main/java/org/apache/aurora/scheduler/state/StateManagerImpl.java
> ffa9481fa4d2f5e1cf74f5819b398e3cb74f1010
> src/main/java/org/apache/aurora/scheduler/storage/TaskStore.java
> 3be2993262cd93f9a847cba8e18bc5f13a91e0bb
>
> src/main/java/org/apache/aurora/scheduler/storage/db/MigrationManagerImpl.java
> ca11e592a1ecf51d47855818b6ee5d7fbc4613a1
> src/main/java/org/apache/aurora/scheduler/storage/log/EntrySerializer.java
> dea2bdab1e8dfc5de950a53f5bd3fb98b1b0aa9a
>
> src/main/java/org/apache/aurora/scheduler/storage/log/SnapshotStoreImpl.java
> b9b3eb0aad3b53ba098cf74a9f874698c9a55226
>
> src/main/java/org/apache/aurora/scheduler/thrift/aop/LoggingInterceptor.java
> d6fbc6ad4f3160e9a015abec39af684175c12b8d
>
> src/main/java/org/apache/aurora/scheduler/updater/JobUpdateControllerImpl.java
> 25b3f37d0ae8adffac21319795b4d2adf73ba40c
> src/test/java/org/apache/aurora/scheduler/storage/log/LogManagerTest.java
> 73440517cddda643c0b84cc04cb8463cdea2da28
>
> Diff: https://reviews.apache.org/r/53796/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Zameer Manji
>
>
