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 2021/05/11 07:14:23 UTC

[GitHub] [airflow] potiuk edited a comment on issue #15770: google provider requires leveldb installed in system

potiuk edited a comment on issue #15770:
URL: https://github.com/apache/airflow/issues/15770#issuecomment-837968561


   I think it's a good idea. We already have the possibility of adding optional dependencies in providers, so it should be rather easy thing to do maybe you would like to add it @dstandish ?
    See: https://github.com/apache/airflow/blob/996965aad9874e9c6dad0a1f147d779adc462278/airflow/providers/apache/beam/provider.yaml#L46
   
   And 
   
   https://github.com/apache/airflow/blob/996965aad9874e9c6dad0a1f147d779adc462278/airflow/providers/google/provider.yaml#L750
   
   It will be just a matter of adding "plyvel" additional extra and moving plyvel out of the 'google'  extra dependencies (+ some documentation)..
   


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