liferoad opened a new pull request, #37857: URL: https://github.com/apache/beam/pull/37857
## What Fixes https://github.com/apache/beam/issues/37833 `apache_beam.transforms.util` was using the root logger (`logging.info(...)`, `logging.warning(...)`, etc.) instead of a module-level named logger. This prevented `sdk_harness_log_level_overrides` from controlling log levels for this module (e.g., suppressing the verbose `BatchElements statistics` message at line 1029). ## Changes - Added `_LOGGER = logging.getLogger(__name__)` after imports - Replaced all 8 root logger calls with `_LOGGER` calls This is consistent with other modules like `apache_beam.transforms.core` which already use `_LOGGER = logging.getLogger(__name__)`. ## How it was tested Minimal change — only replaces the logger object used. No behavioral change unless `sdk_harness_log_level_overrides` is configured, in which case named logger overrides will now correctly apply to `apache_beam.transforms.util`. --- Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily: - [x] Addresses an existing issue: https://github.com/apache/beam/issues/37833 - [x] Includes tests (not needed — no logic change) - [x] Updates documentation (not needed) -- 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]
