dstandish commented on PR #22253: URL: https://github.com/apache/airflow/pull/22253#issuecomment-1100233030
> This is far more important issue to solve than just deprecating config parameters. And the root cause of it is that unlike in all other providers we have functionality split betweeen "core" and "provider" and one uses the other and there is implicit dependency. @potiuk what you are talking about is code sharing, provider depending on code in core, having deps intermixed with core deps. but just because the operator depends on core doesn't mean that it has to take it's configuration in a non-standard way, in a way that no other operator does, namely from `airflow.cfg`. KPO can depend on core _code_ without taking its _configuration_ from core settings. Indeed every provider depends on core -- that's why we have "min airflow version". But what's different with KPO is that it doesn't use a hook, and my goal is to fix that, and have it be configured like every other operator -- with a connection or params. -- 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]
