[GitHub] [airflow] potiuk commented on issue #23059: Presto hook is broken in the latest provider release (2.2.0)

[GitBox <gi...@apache.org>]

potiuk commented on issue #23059:
URL: https://github.com/apache/airflow/issues/23059#issuecomment-1102103559

   > @eladkal good point. I remember there is a CI check for the compatibility. not sure why it did not catch it
   > 
   > @eladkal could you please take a look? #23061
   
   The check is not 100% - it only checks that imports are not broken. We are adding more "back-compatibility" checks as pre-commits but I think we are just learning on the kinds of things that should be taken into account.
   
   Here is the currenet check and we should add try_number check there too: https://github.com/apache/airflow/blob/main/scripts/ci/pre_commit/pre_commit_check_2_1_compatibility.py
   
   In short what needs to happen - we need to formalize the provider's interface a bit more than we did so far in the future and make sure that any changes to "contract" in Airflow are "flagged" and any changes are protected against those changes. For now it is more retro-active but before we split off providers from the core repositry (I think this is something we are going to discuss during the next months) we will have to take the learnings from such cases and formalize it. 
   
   The May 21 bumping min-compatibility to 2.2. will be a good opportunity to do it.
   
   Those cases are a good learning grounds for the future more "robust" solution.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org