moomindani commented on pull request #28953: URL: https://github.com/apache/spark/pull/28953#issuecomment-687959216
@gatorsmile Thank you for your comment. I understand that this kind of hooks are not specific to JDBC, it can be general to all the data sources. However, for JDBC, it should be executed in data source side. This part can be specific to JDBC (or other data sources which include execution engine.). Of course we can have discussion in DSV2 API design, however, can we proceed this PR separately? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org