Hello Dan Burkert, Grant Henke,

I'd like you to do a code review. Please visit

    http://gerrit.cloudera.org:8080/8249

to review the following change.


Change subject: KUDU-2188: restore Java 7 compatibility to artifacts built with 
JDK8
......................................................................

KUDU-2188: restore Java 7 compatibility to artifacts built with JDK8

This particular problem surfaced in an OpenJDK bug report [1] (more info
also available at [2]). To address it we can either:
- Set javac's "boot classpath" to JDK7's rt.jar when building with JDK8.
- Ensure that the ConcurrentHashMap is used as a Map when calling keySet().

Given that there's just one instance of this, the latter is way easier than
the former.

1. https://bugs.openjdk.java.net/browse/JDK-8151366
2. https://gist.github.com/AlainODea/1375759b8720a3f9f094

Change-Id: I27f3db32eaba759f2781a211673fe0aaabd3f83d
---
M java/kudu-client/src/main/java/org/apache/kudu/client/Statistics.java
1 file changed, 12 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/8249/1
--
To view, visit http://gerrit.cloudera.org:8080/8249
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I27f3db32eaba759f2781a211673fe0aaabd3f83d
Gerrit-Change-Number: 8249
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>

Reply via email to