curcur edited a comment on pull request #15674:
URL: https://github.com/apache/flink/pull/15674#issuecomment-823200663
@maver1ck Thanks for reporting and fixing this!
1. Would you mind changing the commit-msg from "Fix for FLINK-22311" to
something more informative, like
"[FLINK-22311] [connector/jdbc] ....", same as your PR title
2. I think precheck is good enough, but what do you think we set
`JdbcExecutionOptions`.maxRetries = 0 directly and document it? Up to you, if
you fix the check style failure, I can accept the diff.
You can do `mvn spotless:apply` to fix most of the check style problem.
--
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]
