Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10324 )
Change subject: [java] Fix TestUtils on Java 9+ ...................................................................... [java] Fix TestUtils on Java 9+ Java 9 made some improvments to the process API causing expected classnames to change and break TestUtils.pidOfProcess. This change optimistically tries to access the pid field of any Process passed to TestUtils.pidOfProcess and throws an exception if the field could not be found or accessed. Change-Id: Ied4ffdca10dc5a0cdc0516964bf6202fd3871f30 Reviewed-on: http://gerrit.cloudera.org:8080/10324 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M java/kudu-client/src/test/java/org/apache/kudu/client/TestUtils.java 1 file changed, 9 insertions(+), 8 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/10324 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ied4ffdca10dc5a0cdc0516964bf6202fd3871f30 Gerrit-Change-Number: 10324 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins
