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:03 UTC
[GitHub] [airflow] potiuk commented on issue #15770: google provider requires leveldb installed in system
potiuk commented 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 @dstandish 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