potiuk edited a comment on issue #15241: URL: https://github.com/apache/airflow/issues/15241#issuecomment-818051410
> Hey there, I'm hoping to test my changes to the dataproc operators sometime today or tomorrow. The CNCF changes are harder to test as the issue in question only surfaces occasionally and only when running large workflows on a kubernetes cluster. In [the original PR](https://github.com/apache/airflow/pull/15137) I did a pretty thorough test, I hope this will suffice. Sure that's perfect. I am going to release it as is. > While setting up my local environment to test the google provider, the initial installation broke because of the introduction of `plyvel` (added in https://github.com/apache/airflow/pull/14105/files) which depends on a preexisting `leveldb` installation. Very Good point. Usually we do not consider "dependency" changes as breaking (you generally find out when you try upgrade whether this is a problem or not). For the only "production" supported environment (linux) `plyvel` has binary releases available for manylinux: https://pypi.org/project/plyvel/#files so it should "just work" for those. This is a major release with new feature/new dependency clearly marked in the documentation, so this should be fine. Thanks for pointing it out ! -- 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: [email protected]
