-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45821/#review127389
-----------------------------------------------------------
Master (ddbb965) is red with this patch.
./build-support/jenkins/build.sh
:commons-args:compileJava
:commons-args:processResources
:commons-args:classes
:commons-args:jar
:commons:compileJavaNote: Writing
file:/home/jenkins/jenkins-slave/workspace/AuroraBot/commons/dist/classes/main/org/apache/aurora/common/args/apt/cmdline.arg.info.txt.1
Note: Writing
file:/home/jenkins/jenkins-slave/workspace/AuroraBot/commons/dist/classes/main/META-INF/compiler/resource-mappings/org.apache.aurora.common.args.apt.CmdLineProcessor
Note:
/home/jenkins/jenkins-slave/workspace/AuroraBot/commons/src/main/java/org/apache/aurora/common/testing/easymock/EasyMockTest.java
uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
:commons:generateThriftResources
:commons:processResources
:commons:classes
:commons:jar
:compileJava/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/storage/db/MybatisCacheImpl.java:29:
error: class MyBatisCacheImpl is public, should be declared in a file named
MyBatisCacheImpl.java
public class MyBatisCacheImpl implements Cache {
^
/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/storage/log/WriteAheadStorage.java:82:
error: cannot find symbol
class WriteAheadStorage extends WriteAheadStorageForwarder implements
^
symbol: class WriteAheadStorageForwarder
/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/storage/log/WriteAheadStorage.java:82:
error: WriteAheadStorage is not abstract and does not override abstract method
fetchFrameworkId() in SchedulerStore
class WriteAheadStorage extends WriteAheadStorageForwarder implements
^
FAILED
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':compileJava'.
> 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 36.705 secs
I will refresh this build result if you post a review containing "@ReviewBot
retry"
- Aurora ReviewBot
On April 6, 2016, 5:48 p.m., Zameer Manji wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45821/
> -----------------------------------------------------------
>
> (Updated April 6, 2016, 5:48 p.m.)
>
>
> Review request for Aurora, Maxim Khutornenko and Bill Farner.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> This swaps out our use of the mybatis default cache with a custom one backed
> by Guava. The benefit of this is that we can now plumb cache statistics out
> which is helpful in debugging performance problems with MyBatis.
>
>
> Diffs
> -----
>
> src/main/java/org/apache/aurora/scheduler/storage/db/MybatisCacheImpl.java
> PRE-CREATION
>
> src/main/resources/org/apache/aurora/scheduler/storage/db/AttributeMapper.xml
> 41519dee391a37ff4904b6c52dc276d278b85f1f
>
> src/main/resources/org/apache/aurora/scheduler/storage/db/TaskConfigMapper.xml
> fd272ccf9b1cfccd9198d1e5e0db37d23f546afa
> src/main/resources/org/apache/aurora/scheduler/storage/db/TaskMapper.xml
> 0219bf3da01936824154675ac812bc53cb308d3f
>
> Diff: https://reviews.apache.org/r/45821/diff/
>
>
> Testing
> -------
>
> Saw metrics in vagrant environment.
>
>
> Thanks,
>
> Zameer Manji
>
>
