marmot-z commented on PR #2475: URL: https://github.com/apache/shardingsphere-elasticjob/pull/2475#issuecomment-3271094342
> > > The failure of unit tests under GraalVM Native Image is not related to the current PR, I have opened #2476 . > > > > > > What else do you need me to do? > > * There's nothing you need to do right now. If you don't mind me reviewing this PR tomorrow, I need to fix the failing unit tests on the master branch first.🤣 > * No one had told me before that the nativeTest branch of the elasticjob master branch was broken because the relevant unit tests were not always triggered by CI and few developers used GraalVM. This is why I previously recommended that you put your unit tests in a folder like `bootstrap/src/test/java/org/apache/shardingsphere/elasticjob/bootstrap`. > * I also found that elasticjob is currently not compilable with JDK 24, but this does not affect the current PR. I will open an issue later to investigate what happened. Ok, Thank you for your patient review and selfless dedication 😄 👍 -- 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]
