Hello Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4425
to look at the new patch set (#2).
Change subject: java: enable error-prone for java builds
......................................................................
java: enable error-prone for java builds
error-prone (http://errorprone.info/) is a compiler extension by Google
which checks for a bunch of common Java errors. One of the most useful
features is that it verifies JCIP annotations like GuardedBy.
This includes some fixes to get it to pass. We should break these fixes
out into separate commits.
It also seems that building kudu-client with error-prone is extremely
slow, probably due to the generated code. We should figure out how to
avoid running it on the protobuf code.
Change-Id: Iebafbb307904875490426dd699ab1f55e21d284d
---
M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java
M java/kudu-client/src/main/java/org/apache/kudu/client/PartitionPruner.java
M java/kudu-client/src/main/java/org/apache/kudu/client/RowError.java
M java/kudu-client/src/main/java/org/apache/kudu/client/Statistics.java
M java/kudu-client/src/main/java/org/apache/kudu/client/TableLocationsCache.java
M java/pom.xml
6 files changed, 30 insertions(+), 11 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/25/4425/2
--
To view, visit http://gerrit.cloudera.org:8080/4425
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iebafbb307904875490426dd699ab1f55e21d284d
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
