wchevreuil commented on code in PR #157:
URL: https://github.com/apache/hbase-connectors/pull/157#discussion_r3536684638
##########
dev-support/jenkins/Dockerfile:
##########
@@ -28,6 +28,18 @@ RUN apt-get -q update && apt-get -q install
--no-install-recommends -y \
apt-get clean && \
rm -rf /var/lib/apt/lists/*
+# JDK 17 for the Spark4 profile (maven-enforcer-plugin requires JDK 17+)
+RUN wget -q
https://github.com/adoptium/temurin17-binaries/releases/download/jdk-17.0.11%2B9/OpenJDK17U-jdk_x64_linux_hotspot_17.0.11_9.tar.gz
-O /tmp/jdk17.tar.gz && \
+ mkdir -p /usr/local/openjdk-17 && \
+ tar -xzf /tmp/jdk17.tar.gz --strip-components=1 -C /usr/local/openjdk-17
&& \
+ rm /tmp/jdk17.tar.gz
+
+# Yetus xml check uses jrunscript (removed in JDK 14). Provide a wrapper so
+# it remains available when JAVA_HOME points to JDK 17.
+RUN printf '#!/bin/sh\nexec /usr/local/openjdk-8/bin/jrunscript "$@"\n' \
Review Comment:
Good point. Let me check how does yetus is configured with jdk 17+. Here, I
was getting a pom.xml parsing issue, and yetus was then reporting an xml error.
The suggestion I got from claude-opus-4-6 was that jdk17 misss this jrunscript.
I think the main problem here is that we are trying to support both jdk8 and
jdk17.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
