You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/05/30 10:37:55 UTC

[GitHub] [airflow] potiuk commented on pull request #9068: Make airflow/macros/hive.py Pylint compatible

potiuk commented on pull request #9068:
URL: https://github.com/apache/airflow/pull/9068#issuecomment-636312271


   Might be a good idea but our deprecation policy (maybe not fully documented yet) is that we should deprecate rather than remove immediately. Though I agree some of the code might not be useful at all.
   
   @KevinYang21 I believe you were adding/modifying it in the past. Do you think this belongs to core? Or is it safe to remove it or should we deprecate or maybe move to "apache.hive" provider? Is it generic enough to be useful for others? 


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

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