aglinxinyuan commented on code in PR #6880:
URL: https://github.com/apache/texera/pull/6880#discussion_r3671274850
##########
amber/src/main/python/texera_run_python_worker.py:
##########
@@ -18,8 +18,19 @@
import base64
import json
import sys
+import warnings
from loguru import logger
+# `fs` (imported transitively by the core import below) pulls in pkg_resources
+# at import time (#4199), which emits this deprecation warning on stderr in
+# every spawned worker. It is not actionable on our side, so filter it before
+# the import chain runs. Mirrored for pytest in amber/pyproject.toml.
+warnings.filterwarnings(
Review Comment:
Good point on the debt going quiet — that tracking issue does exist: **#6917
("Migrate off unmaintained PyFilesystem2 (fs) and unpin setuptools")**, still
open. It covers both halves you mention: dropping the EOL `fs` 2.4.16
dependency and lifting the `setuptools<82` pin once `pkg_resources` is gone.
I've linked it from the filter comment so the pointer lives next to the
suppression rather than only in this thread. For context, #6928 was a first
attempt at the removal (stdlib `tempfile`); it's closed now, so #6917 remains
the live tracker.
--
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]
