sfc-gh-madkins commented on pull request #16420: URL: https://github.com/apache/airflow/pull/16420#issuecomment-860868813
Yes -- The hook is where it should happen where the param is unchangeable by the consumer. For managed providers, they can then override it when they go to push out their version of Airflow. On Mon, Jun 14, 2021 at 12:01 PM Jarek Potiuk ***@***.***> wrote: > I think, it should be connection parameter, environment parameter or fixed > constant (preferred by me). Now we require changes to the DAG file, so very > few people will be using this feature. > > It's added to Hook (and bubbled up to Operator). I think indeed it would > be better to get it as extra in connection with 'AIRFLOW' default. WDYT > @sfc-gh-madkins <https://github.com/sfc-gh-madkins> ? > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/airflow/pull/16420#issuecomment-860841495>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/ATSRCU4XEVCOHPEW23SGTETTSYYU3ANCNFSM46TNNUEQ> > . > -- 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]
