claudevdm commented on PR #34417: URL: https://github.com/apache/beam/pull/34417#issuecomment-2758282201
> Thanks, this is looking good! Just one thing to add: Integration tests [here](https://github.com/apache/beam/blob/master/sdks/java/io/jdbc/src/test/java/org/apache/beam/sdk/io/jdbc/JdbcIOIT.java#L242). @svetakvsundhar , there are no changes to JdbcIO in this PR, what kind of integration tests are you thinking of? -- 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: github-unsubscr...@beam.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
