difin commented on code in PR #6327:
URL: https://github.com/apache/hive/pull/6327#discussion_r2880432756


##########
standalone-metastore/metastore-rest-catalog/pom.xml:
##########
@@ -19,12 +19,97 @@
   <modelVersion>4.0.0</modelVersion>
   <artifactId>hive-standalone-metastore-rest-catalog</artifactId>
   <name>Hive Metastore REST Catalog</name>
+  <repositories>
+    <repository>
+      <id>central</id>
+      <url>https://repo.maven.apache.org/maven2</url>
+      <releases>
+        <enabled>true</enabled>
+      </releases>
+      <snapshots>
+        <enabled>false</enabled>
+      </snapshots>
+    </repository>
+  </repositories>
   <properties>
     <standalone.metastore.path.to.root>..</standalone.metastore.path.to.root>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     <log4j2.debug>false</log4j2.debug>
     <iceberg.version>1.10.0</iceberg.version>
+    <spring-boot.version>2.7.18</spring-boot.version>
+    <!-- Override Spring Boot's Jetty 9.4.53 with Hive's 9.4.57 to avoid 
version conflicts -->
+    <jetty.version>9.4.57.v20241219</jetty.version>
   </properties>
+  <dependencyManagement>
+    <dependencies>
+      <!-- Align all Jetty artifacts to Hive's version; Spring Boot 2.7.18 
defaults to 9.4.53 -->

Review Comment:
   This is a much broader upgrade.
   Currently Hive uses Spring 5.3.39, Jetty 9.4.57.v20241219 and servlet API 
4.0.
   Spring boot 2.7.18 is compatible with Spring 5.3.39.
   Moving to Spring Boot 3.0+ requires Spring 6.0+, Jetty 11+, Servlet API 5.0.



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to