[GitHub] [pinot] walterddr commented on a diff in pull request #8991: update-dependencies

[GitBox <gi...@apache.org>]

walterddr commented on code in PR #8991:
URL: https://github.com/apache/pinot/pull/8991#discussion_r910440518


##########
pom.xml:
##########
@@ -121,24 +121,24 @@
       See also: the surefire plugin section and the profiles section.-->
     <SKIP_INTEGRATION_TESTS>true</SKIP_INTEGRATION_TESTS>
     <!-- Configuration for unit/integration tests section 1 of 3 (properties) ENDS HERE.-->
-    <avro.version>1.9.2</avro.version>
-    <parquet.version>1.11.1</parquet.version>
-    <helix.version>0.9.8</helix.version>
-    <zkclient.version>0.7</zkclient.version>
-    <jackson.version>2.10.0</jackson.version>
-    <zookeeper.version>3.5.8</zookeeper.version>
+    <avro.version>1.11.0</avro.version>
+    <parquet.version>1.12.3</parquet.version>
+    <helix.version>1.0.4</helix.version>
+    <zkclient.version>0.11</zkclient.version>
+    <jackson.version>2.13.3</jackson.version>
+    <zookeeper.version>3.5.9</zookeeper.version> <!-- 3.7.1 -->
     <async-http-client.version>2.12.3</async-http-client.version>
     <jersey.version>2.35</jersey.version>
     <grizzly.version>2.4.4</grizzly.version>
     <hk2.version>2.6.1</hk2.version>
-    <swagger.version>1.5.16</swagger.version>
-    <hadoop.version>2.10.1</hadoop.version>
+    <swagger.version>1.6.6</swagger.version>
+    <hadoop.version>2.10.2</hadoop.version>
     <antlr.version>4.6</antlr.version>
     <jsonpath.version>2.7.0</jsonpath.version>
     <quartz.version>2.3.2</quartz.version>
-    <calcite.version>1.29.0</calcite.version>
+    <calcite.version>1.30.0</calcite.version>
     <lucene.version>8.2.0</lucene.version>
-    <reflections.version>0.9.9</reflections.version>
+    <reflections.version>0.10.2</reflections.version>

Review Comment:
   this is a lot of upgrades at the same time. especially there's also 
   - helix major version bump 0.9.8 --> 1.0.4
   - calcite version bump usually result in some interesting compilation errors
   
   I would suggest break down this into smaller upgrade PRs 



-- 
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: commits-unsubscribe@pinot.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org