Adar Dembo has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/12816 )
Change subject: KUDU-2099: drop Java 7 support
......................................................................
KUDU-2099: drop Java 7 support
Two commits merged since 1.9 yield compilation errors when building with
JDK 10 or later:
.../client/AuthzTokenCache.java:230: error: cannot find symbol
List<RpcAndException> pendingRetries = retriesForTable.putIfAbsent(
^
symbol: method putIfAbsent(String,List<RpcAndException>)
location: variable retriesForTable of type Map<String,List<RpcAndException>>
.../client/KuduScannerIterator.java:34: error: KuduScannerIterator is not
abstract and does not override abstract method remove() in Iterator
public class KuduScannerIterator implements Iterator<RowResult> {
^
The root cause is that with JDK 10, these two methods have been lifted out
of their respective classes and are now defined via default interface
implementations, but the compiler can't emit those implementations into
Java 7-compatible bytecode[1].
So we can either work around those two issues, or we can drop support for
Java 7. Let's see if there are any objections to the latter.
1. https://stackoverflow.com/a/23757061
Change-Id: I05fcea8bbc7b773f95429b324acd0b9d18a6e9f8
Reviewed-on: http://gerrit.cloudera.org:8080/12816
Tested-by: Kudu Jenkins
Reviewed-by: Andrew Wong <aw...@cloudera.com>
Reviewed-by: Grant Henke <granthe...@apache.org>
Reviewed-by: Todd Lipcon <t...@apache.org>
---
M java/README.adoc
M java/gradle.properties
M java/gradle/dependencies.gradle
3 files changed, 3 insertions(+), 8 deletions(-)
Approvals:
Kudu Jenkins: Verified
Andrew Wong: Looks good to me, but someone else must approve
Grant Henke: Looks good to me, approved
Todd Lipcon: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.cloudera.org:8080/12816
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I05fcea8bbc7b773f95429b324acd0b9d18a6e9f8
Gerrit-Change-Number: 12816
Gerrit-PatchSet: 4
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gsolov...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
