potiuk commented on code in PR #31920:
URL: https://github.com/apache/airflow/pull/31920#discussion_r1231020322
##########
docker_tests/test_prod_image.py:
##########
@@ -127,7 +127,7 @@ def test_pip_dependencies_conflict(self):
"cncf.kubernetes": ["kubernetes", "cryptography"],
"dask": ["cloudpickle", "distributed"],
"docker": ["docker"],
- "elasticsearch": ["elasticsearch", "es.elastic", "elasticsearch_dsl"],
+ "elasticsearch": ["elasticsearch"],
Review Comment:
Those are the kinds of problems I mentioned when "experimenting". It would
have have to be extracted and merged before, effectively making dependencies
between PRs. Easier for release manager, more difficult for contributor.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
