gseva opened a new pull request #4080: [AIRFLOW-XXX] Make hmsclient import 
optional
URL: https://github.com/apache/incubator-airflow/pull/4080
 
 
   
   ### Jira
   
   - No jira issue
   
   ### Description
   
   Currently to use anything from hive_hooks.py you must have hmsclient 
installed, which is inconsistent: thrift imports are made inside the 
`get_metastore_client` method, and hnsclient is imported outside (and it does 
thrift imports internally).
   
   ### Tests
   
   Not sure if tests needed for this change.
   
   ### Documentation
   
   ### Code Quality
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to