potiuk commented on PR #28538:
URL: https://github.com/apache/airflow/pull/28538#issuecomment-1364091766

   > It is tricky though usualy we deprecate first but I'm not sure if there is 
a way to raise deprecation warning for macros?
   
   Technically it's not "deprecation" - it should work identically to olde way 
but only when the provider is installed. This is yet another case where things 
are not breaking - as long as we upgrade dependencies.
   
   We could - potentially  raise a warning here - it but not just yet. After 
releasing Hive Provider 5.1.0 (but only after that, because otherwise the user 
would not have a way to get rid of the warning) we could raise such warnings - 
but this will have to be in Airflow 2.5.n "airflow.macros.hive" - methods - and 
the way to deprecate the warnings would be to ask the user to upgrade the Hive 
Provider to 5.1.0+. It will have to be done directly in v2-5-test branch though.
   


-- 
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]

Reply via email to