lazyfetch commented on PR #11012: URL: https://github.com/apache/seatunnel/pull/11012#issuecomment-4813952116
Hi @DanielLeens , the Sybase source is successfully reading data (local testing confirms all test rows are processed successfully). However, the E2E test fails locally during the final cleanup phase. As seen in the logs, ChunkSplitter.close() triggers SimpleJdbcConnectionProvider.isConnectionValid(), which hardcodes a call to connection.isValid(). Because the standard jTDS driver for Sybase is JDBC 3.0 and does not implement isValid(), it throws an AbstractMethodError and crashes the test teardown. Since ChunkSplitter uses SimpleJdbcConnectionProvider directly, I cannot bypass this using dialect overrides without stepping out of the scope of this source-connector PR. Should I submit a defensive try/catch for AbstractMethodError in SimpleJdbcConnectionProvider.java as part of this PR to protect the framework from legacy drivers, or is there a preferred way you'd like me to handle this known driver limitation for the E2E test? or am i just getting something wrong here? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
