damccorm commented on code in PR #35297:
URL: https://github.com/apache/beam/pull/35297#discussion_r2150080688
##########
sdks/python/setup.py:
##########
@@ -378,7 +378,6 @@ def get_portability_package_data():
'numpy>=1.14.3,<2.3.0', # Update pyproject.toml as well.
'objsize>=0.6.1,<0.8.0',
'packaging>=22.0',
- 'pymongo>=3.8.0,<5.0.0',
Review Comment:
I like this change, thanks. Up to you on whether you want to take this on,
but I think we would benefit from doing this with more dependencies.
Specifically, the following are used very lightly and are not needed for core
functionality:
- hdfs (would need a new extra)
- jsonschema (just used in yaml, could be moved to that extra)
- pydot (just used in interactive, could be moved to that extra)
- redis
> I also feel that we should update the
[CHANGES.md](https://github.com/apache/beam/blob/master/CHANGES.md) file to
mention this change.
It is breaking, so I agree - could you please do this as part of this PR?
--
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: github-unsubscr...@beam.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
