owenmonn commented on code in PR #3539:
URL: https://github.com/apache/amoro/pull/3539#discussion_r2065845144


##########
pom.xml:
##########
@@ -1368,7 +1368,7 @@
             <id>hadoop2</id>
             <properties>
                 <guava.version>14.0.1</guava.version>
-                <hive.version>2.3.9</hive.version>
+                <hive.version>2.3.8</hive.version>

Review Comment:
   Thanks for your review.
   
   > How about adding a comment here to tell why we are not using the earliest 
2.3 version?
   
   To clarify why I didn't choose the earliest 2.3 version, it wasn't due to 
any specific stability concerns I had for other 2.3.x versions. I was mainly 
trying to work around the 2.3.9 issue while keeping the Hive dependency 
relatively up-to-date within the 2.3 range. I'd be interested to hear if you 
have any information about the stability of particular 2.3.x versions.
   
   On a related note, I was also wondering if the potential impact of this 
change on compatibility with Hive Metastore versions prior to 2.3 has been 
considered? I just wanted to ensure we're taking all compatibility aspects into 
account.



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

Reply via email to