tamirsagi commented on PR #603:
URL: 
https://github.com/apache/flink-kubernetes-operator/pull/603#issuecomment-1552842225

   > Please run `mvn clean install` locally before you push to avoid the need 
to run the CI again and again. There is still a spotless formatting issue
   
   Hey Gyula,
   
   I ran it locally prior MR, there were policy errors that are not related to 
my changes. for instance
   com.diffplug.spotless.FormatExceptionPolicyLegacy error
   SEVERE: Step 'removeUnusedImports' found problem in 
'src\main\java\org\apache\flink\kubernetes\operator\kubeclient\decorators\CmdStandaloneJobManagerDecorator.java':null
   Caused by: java.lang.IllegalAccessError: class 
com.google.googlejavaformat.java.RemoveUnusedImports (in unnamed module 
@0x70a984c5) cannot access class com.sun.tools.javac.util.Context (in module 
jdk.compiler) because module jdk.compiler does not export 
com.sun.tools.javac.util to unnamed module @0x70a984c5
   
   I believe it's related to that fact that JDK 17 is being used.
   
   I added a simple null check and 2 tests and got errors regarding format 
issues (First import order and now something else) . I checked the failure in 
CI but it was not clear what exactly the violation is. A little assistance with 
the format error would be greatly appreciated. 
   
   
   


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to