1996fanrui commented on PR #929: URL: https://github.com/apache/flink-kubernetes-operator/pull/929#issuecomment-2611886843
> @1996fanrui It appears that MySQL/Postgresql DB restart test cases are failing in CI with SQLTransientException. Let me update the test to ignore transient exception as typically subsequent connection should be successful. Before this PR, the test case produces SQLNonTransientConnectionException. We may also consider adding retries but I feel it's overkill. lmk your thoughts After I debug it on my Mac, I don't think catch `SQLTransientException` is reasonable. Please check the following figure, the `bbbbb` is not executed. It means we cannot ensure the connection works after restarting. So I think retry mechanism is needed, we need to retry queryData from data base until the result is expected. Also, I added the retry mechanism on my Mac, but it still doesn't work, I found after restart, the port of MySQL Container is changed. It means the original data source won't be used.  -- 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
