chesnokoff commented on PR #13732:
URL: https://github.com/apache/ignite/pull/13732#issuecomment-6409899050

   Thanks for the new version, looks good to me. I had an idea to add to 
parent/pom.xml:
   
   ```xml
   <requireJavaVersion>
       <version>[17.0.11,)</version>
       <message>Building Ignite requires JDK 17.0.11 or later: javac must 
accept -proc:full.</message>
   </requireJavaVersion>
   ```
   But looks like some of our ci test agents (Check Javadoc step) use < 17.0.11 
so the rule can be added later. For now I run the patch on teamcity
   
   Also, a small correction: -proc:full was backported to 17.0.11 and 11.0.23 
(JDK-8324670, JDK-8324804), so the unsupported range is 17.0.0–17.0.10.


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

Reply via email to