dang-stripe commented on code in PR #11799:
URL: https://github.com/apache/pinot/pull/11799#discussion_r1366190037
##########
pinot-plugins/pinot-file-system/pinot-s3/pom.xml:
##########
@@ -39,6 +39,7 @@
<http.core.version>4.4.13</http.core.version>
<s3mock.version>2.12.2</s3mock.version>
<javax.version>3.1.0</javax.version>
+ <woodstox.version>5.3.0</woodstox.version>
Review Comment:
👍 i bumped it to 6.4.0. the release docs say 6.x are all backward compatible
w/ 5.x: https://github.com/FasterXML/woodstox/wiki/Woodstox-Release-6.0
> Woodstox 6.0 is the first Java 9 module info compatible version: it is,
however, fully API-compatible 5.x and does not require newer JDK version either.
i can also re-test this again on one of our clusters.
##########
pinot-plugins/pinot-file-system/pinot-s3/pom.xml:
##########
@@ -39,6 +39,7 @@
<http.core.version>4.4.13</http.core.version>
<s3mock.version>2.12.2</s3mock.version>
<javax.version>3.1.0</javax.version>
+ <woodstox.version>5.3.0</woodstox.version>
Review Comment:
👍 i bumped it to 6.4.0. the release docs say 6.x are all backward compatible
w/ 5.x: https://github.com/FasterXML/woodstox/wiki/Woodstox-Release-6.0
> Woodstox 6.0 is the first Java 9 module info compatible version: it is,
however, fully API-compatible 5.x and does not require newer JDK version either.
i can also re-test this again on one of our clusters.
--
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]
