[GitHub] [hadoop] jojochuang commented on a diff in pull request #5345: HADOOP-18613. Upgrade ZooKeeper to version 3.8.1

["jojochuang (via GitHub)" <gi...@apache.org>]

jojochuang commented on code in PR #5345:
URL: https://github.com/apache/hadoop/pull/5345#discussion_r1200819683


##########
hadoop-project/pom.xml:
##########
@@ -37,7 +37,7 @@
     <!--Whether to proceed to next module if any test failures exist-->
     <maven.test.failure.ignore>true</maven.test.failure.ignore>
     <maven.test.redirectTestOutputToFile>true</maven.test.redirectTestOutputToFile>
-    <jetty.version>9.4.48.v20220622</jetty.version>
+    <jetty.version>9.4.49.v20220914</jetty.version>

Review Comment:
   This is unrelated. Are you updating Jetty because Zookeeper has jetty dependency and you're trying to harmonize them?
   
   If so I'd suggest to exclude the jetty from zookeeper. We update Jetty frequently and do not necessarily use the same version as Zookeeper.



-- 
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: common-issues-unsubscribe@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org