dstandish commented on PR #24737: URL: https://github.com/apache/airflow/pull/24737#issuecomment-1170386373
seems like a good idea but think you need to think about how to incorporate this support for the Connection object. if you can do it through hook init params, probably you should be able to configure it through connection. in connection i've seen it done two ways. either (1) you forward anything in extra (besides maybe a list of reserved params) to client as kwargs or (2) you add a `client_kwargs` dict within extra. wdyt? -- 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]
