potiuk commented on PR #28538: URL: https://github.com/apache/airflow/pull/28538#issuecomment-1364058665
Ok . I tested it and it works well now. Including installing the new provider on earlier version of airflow that has `hive` macros built-in and the macros from the provider nicely override the hive macros that come from Airflow. I've added >= 5.1.0 exception for Hive in airflow's `[hive]` extra so we can be sure that the new version of Airflow pulls the new version of hive when installed with extrra. The only potentially dangerous scenario is when someone will upgrade just airflow to > 2.6.0 (I assume this change will be out in 2.6.0) and still has the old hive provider installed. This should geenerally not happen, because of constraints and it has an easy upgrede mechanism if it happens (I added newsfragment describing it). -- 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]
