It seems like the current version of maven-java-formatter-plugin that we depend on is adding in trailing whitespace to fate/**/ZooCache.java which is then causing the checkstyle verification to fail the build.

An example: https://secure.penguinsinabox.com/jenkins/job/Accumulo-Integration-Tests/359/console

The repository doesn't contain any trailing whitespace on L239, yet on my build machine:

diff --git a/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooCache.java b/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooCache.java
index 8c9f80d..33e5261 100644
--- a/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooCache.java
+++ b/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooCache.java
@@ -290,7 +290,7 @@ public class ZooCache {
         /*
* The following call to exists() is important, since we are caching that a node does not exist. Once the node comes into existence, it will be added to * the cache. But this notification of a node coming into existence will only be given if exists() was previously called.
-         *
+         *
* If the call to exists() is bypassed and only getData() is called with a special case that looks for Code.NONODE in the KeeperException, then
          * non-existence can not be cached.
          */

@Christopher, is this a bug in your plugin? Maybe the underlying formatter itself?

Reply via email to