potiuk commented on PR #28538: URL: https://github.com/apache/airflow/pull/28538#issuecomment-1363137352
I am not sure (tired?) I thought the "airflow.macros" macros also belong to hive and I moved them together with hive ones. I believe with the approach I proposed now - we do not even need to add any deprecations. If we properly expose the hive macros via additional entrypoint in the hive provider, we can just move "macros.hive" entrely to the provider - because users interact with them through JINJA only an the macros will be available as `hive.max_partition` for example. I think the only thing left (besides testing of course) is to make sure that we document this as potentially breaking change in the docs (users will need to upgrade hive provider) and add >= <new_provider_version> in the hive extra for airlfow (and the docs of course mentioned by @eladkal ). -- 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]
