Abacn commented on PR #34394:
URL: https://github.com/apache/beam/pull/34394#issuecomment-2749098821

   > we can just change the JdbcIO.read transform (pointed above, 
JdbcIO.java#L1513) added by jdbcio.readfrompartition to explicitly set a schema 
(Long), this is a much smaller change than introducing parameters, and user 
does not need to set these parameters also
   
   sorry, I see, this is a JdbcIO.read instead of JdbcIO.readRows, so it does 
not need database connection at expansion time. Where is the database 
connection happens currently?
   


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

Reply via email to