krasinski commented on pull request #12462: URL: https://github.com/apache/flink/pull/12462#issuecomment-640573199
Looks good to me. One additional thing comes to my mind: when I started working on the fix to address review comments I've encountered a problem with java modules when using `sun.security.krb5.Config.refresh()` so adding a new line in java 11 profile compiler args in the main `pom.xml` might be needed: `--add-exports=java.security.jgss/sun.security.krb5=ALL-UNNAMED` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
