1996fanrui commented on PR #929:
URL: 
https://github.com/apache/flink-kubernetes-operator/pull/929#issuecomment-2611689010

   > I addressed the remaining comments and added a test case for db restart to 
`AbstractJdbcStateInteractorITCase`. The same could be added to 
`AbstractJdbcEventInteractorITCase` if needed but it's a bit redundant.
   
   Sounds make sense to me, testing restart in 
AbstractJdbcStateInteractorITCase is enough, but the test is failed. 
   
https://github.com/apache/flink-kubernetes-operator/actions/runs/12939940745/job/36097810802?pr=929#step:8:1025
   
   Also, I may need to do another round of overall review after the test passes.


-- 
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