XiaoHongbo-Hope opened a new pull request, #8902: URL: https://github.com/apache/paimon/pull/8902
### Purpose `pypaimon.read.reader.aggregate` eagerly registers built-in aggregators. Importing the registry therefore imported the native `_datasketches` module even when a table did not use `theta_sketch`, preventing ordinary PyPaimon imports and reads in environments without DataSketches. This change: - moves DataSketches from the base dependencies to the `theta-sketch` extra; - imports `_datasketches` only when two non-null theta sketches are merged; - raises an actionable installation error when the optional dependency is required; and - keeps DataSketches installed in the Python 3.7 CI lane that exercises the theta aggregator. ### Tests - `python -m unittest -v pypaimon.tests.test_field_aggregators pypaimon.tests.test_optional_datasketches_dependency` (124 tests) - `flake8 --config=dev/cfg.ini` on the changed Python files and `setup.py` - generated egg metadata confirms `datasketches` appears only under the `theta-sketch` extra -- 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]
