o-nikolas commented on PR #73301:
URL: https://github.com/apache/airflow/pull/73301#issuecomment-5779765198

   > I quite agree with @ashb that we need to at least discuss on how DuckDB 
support should work - and intuitively it does seem like S3 support in DuckDB 
provider seems like a better option.
   
   Hey @potiuk
   
   I already made the changes @ashb was asking for and moved the S3 support 
into the DuckDB provider? Following the same connection driven hook selection 
as we use for for BaseSQLOperator._hook.
   
   If you also agree with Ash, and I've already implemented those changes, is 
there something else you'd like to discuss 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]

Reply via email to