potiuk commented on issue #16340:
URL: https://github.com/apache/airflow/issues/16340#issuecomment-857634303
Option 2 is better but those parameters should be passed as 'extras' of the
connection. Airflow already has support for those parameters to be passed as
extras and they can be passed as json-encoded __extra parameter. You can then
even add UI fields to be able to configure those fields via UI.
See
https://airflow.apache.org/docs/apache-airflow/stable/howto/connection.html#connection-uri-format
and there are a number of other hooks that are using this mechanism (including
extra_dejson method to get those parameter value as json)
--
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:
[email protected]
