cecemei commented on code in PR #18831:
URL: https://github.com/apache/druid/pull/18831#discussion_r2611978914
##########
embedded-tests/pom.xml:
##########
@@ -29,6 +29,7 @@
<properties>
<surefire.rerunFailingTestsCount>0</surefire.rerunFailingTestsCount>
+ <maven.compiler.release>17</maven.compiler.release>
Review Comment:
update the root pom to include things to check for upgrade to 17.
imo using 17 in embedded-test is safe and multi-line string syntax helps
with readability a lot, so hopefully we dont let this java 11 thing drag up
back.
--
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]
