fix 4.0.1 by relocating jline

Author: maltesander

hive/Dockerfile

@@ -47,7 +47,7 @@ RUN /stackable/patchable --images-repo-root=src checkout hive ${PRODUCT_VERSION}
 
 # Use bash for regex machting, otherwise docker lint is complaining:
 # hive/Dockerfile:51 SC3014 warning: In POSIX sh, == in place of = is undefined.
-SHELL ["/bin/bash", "-c"]
+SHELL ["/bin/bash", "-o", "pipefail", "-c"]
 
 # Make expensive maven build a separate layer for better caching
 # Cache mounts are owned by root by default

hive/hive-metastore-opa-authorizer/Dockerfile

@@ -25,7 +25,7 @@ COPY --chown=${STACKABLE_USER_UID}:0 --from=hadoop-builder /stackable/patched-li
 
 # Use bash for regex machting, otherwise docker lint is complaining:
 # hive/hive-metastore-opa-authorizer/Dockerfile:26 SC3015 warning: In POSIX sh, =~ regex matching is undefined.
-SHELL ["/bin/bash", "-c"]
+SHELL ["/bin/bash", "-o", "pipefail", "-c"]
 
 # Make expensive maven build a separate layer for better caching
 # Cache mounts are owned by root by default
@@ -55,10 +55,7 @@ if [[ "${HIVE_VERSION}" =~ "^3" ]]; then
 # Hive 4.0.1 only works with the shaded jar
 elif [[ "${HIVE_VERSION}" == "4.0.1" ]]; then
   mvn clean package -DskipTests -Dhive.version=${HIVE_VERSION} -Dhadoop.version=${HADOOP_VERSION} -f hms-v4/pom.xml
-  # The renaming from "com.bosch.bdps.hms4-..." to "zhms4-.." is required for 4.0.1 because it is a dependency mess and otherwise loaded before other hive
-  # libraries. Setting CLASSPATH or HIVE_AUX_JARS_PATH did not reorder the dependency loading up as well. 
-  # This is very very bad and depends on alhpabetically ordering...
-  mv hms-v4/target/com.bosch.bdps.hms4-${HIVE_VERSION}-${HADOOP_VERSION}-${AUTHORIZER_VERSION}.jar /stackable/opa-authorizer-bin/zhms4-${HIVE_VERSION}-${HADOOP_VERSION}-${AUTHORIZER_VERSION}.jar
+  mv hms-v4/target/com.bosch.bdps.hms4-${HIVE_VERSION}-${HADOOP_VERSION}-${AUTHORIZER_VERSION}.jar /stackable/opa-authorizer-bin/hms4-${HIVE_VERSION}-${HADOOP_VERSION}-${AUTHORIZER_VERSION}.jar
 # Hive 4.1.0 only works with the non shaded jar
 else
   mvn clean package -DskipTests -Dhive.version=${HIVE_VERSION} -Dhadoop.version=${HADOOP_VERSION} -f hms-v4/pom.xml

hive/hive-metastore-opa-authorizer/stackable/patches/v1.0.0/0001-Shade-jline-explicitly.patch

@@ -0,0 +1,39 @@
+From 500af58b1022e65ba3b498e210d6f69fa1e3a1d4 Mon Sep 17 00:00:00 2001
+From: Malte Sander <malte.sander.it@gmail.com>
+Date: Mon, 24 Nov 2025 15:00:03 +0100
+Subject: Shade jline explicitly
+
+---
+ hms-v4/pom.xml | 19 +++++++++++++++++++
+ 1 file changed, 19 insertions(+)
+
+diff --git a/hms-v4/pom.xml b/hms-v4/pom.xml
+index 4ebeac4..e1d6fd9 100644
+--- a/hms-v4/pom.xml
++++ b/hms-v4/pom.xml
+@@ -67,6 +67,25 @@
+                                     </excludes>
+                                 </filter>
+                             </filters>
++                            <!-- 
++                            For Hive 4.0.1 the dependency loading is screwed. We explicitly shade jline to avoid a schematool dependency mess:
++
++                            Exception in thread "main" java.lang.NoSuchMethodError: 'void org.jline.reader.impl.completer.StringsCompleter.<init>(org.jline.reader.Candidate[])'
++                                at sqlline.SqlLineOpts.setOptionCompleters(SqlLineOpts.java:160)
++                                at sqlline.Application.getCommandHandlers(Application.java:294)
++                                at sqlline.SqlLine$Config.<init>(SqlLine.java:1946)
++                                at sqlline.SqlLine.setAppConfig(SqlLine.java:1875)
++                                at sqlline.SqlLine.<init>(SqlLine.java:229)
++                                at org.apache.hadoop.hive.metastore.tools.schematool.MetastoreSchemaTool.execSql(MetastoreSchemaTool.java:313)
++
++                            This does not affect versions >= 4.1.x since the non shaded jar is used.    
++                            -->
++                            <relocations>
++                                <relocation>
++                                    <pattern>org.jline</pattern>
++                                    <shadedPattern>com.bosch.bdps.jline</shadedPattern>
++                                </relocation>
++                            </relocations>
+                         </configuration>
+                     </execution>
+                 </executions>