aglinxinyuan commented on code in PR #6880: URL: https://github.com/apache/texera/pull/6880#discussion_r3654277721
########## 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 call — opened #6917 to track this. Digging into it, the migration turned out smaller than expected: `ExecutorManager` is the only consumer, and fs's `TempFS` is internally just `mkdtemp` + `rmtree`, so a stdlib swap is behavior-preserving. The follow-up PR can also drop the setuptools pin (fs is the only distribution in the closure that needs it) and revert this filter once the warning's source is gone — footprint details are in the issue. -- 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]
