michaelsembwever commented on code in PR #3696:
URL: https://github.com/apache/cassandra/pull/3696#discussion_r1917329596


##########
build.xml:
##########
@@ -45,7 +45,7 @@
         The use of both CASSANDRA_USE_JDK11 and use-jdk11 is deprecated.
     -->
     <property name="java.default" value="11" />
-    <property name="java.supported" value="11,17" />
+    <property name="java.supported" value="11,17,21" />

Review Comment:
   Yes.  My vote is leave it in and let it become part of ci-cassandra 
post-commit CI.
   
   That doesn't mean it has to be mandatory for pre-commit CI (so long as we 
accept we can't yet blame folk for introducing jdk21 breakages).   This aligns 
to what I believe is being proposed here with prematurely introducing 21 
without dropping 11.



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

Reply via email to