karuppuchamysuresh opened a new pull request, #20057:
URL: https://github.com/apache/datafusion/pull/20057
Downgrade the log level for record batch memory usage warnings from warn! to
debug! to reduce production log pollution. This warning was being triggered
excessively in production when record batch memory usage exceeded expected
limits, creating noise without affecting functional correctness.
The diagnostic information remains available at debug level for development
and troubleshooting purposes.
Fixes #19846
## Which issue does this PR close?
<!--
We generally require a GitHub issue to be filed for all bug fixes and
enhancements and this helps us generate change logs for our releases. You can
link an issue to this PR using the GitHub syntax. For example `Closes #123`
indicates that this PR will close issue #123.
-->
- Closes #19846
## Rationale for this change
The warning message about record batch memory usage exceeding expected
limits
was being triggered excessively in production environments, creating log
pollution without affecting functional correctness.
## What changes are included in this PR?
- Changed log level from `warn!` to `debug!` in
`datafusion/physical-plan/src/spill/mod.rs`
- Updated the import from `log::warn` to
`log::debug`
This reduces production noise while keeping the diagnostic information
available at debug level for development and troubleshooting.
## Are these changes tested?
The change is a simple log level adjustment and doesn't affect the logic.
Existing tests continue to validate the functionality.
## Are there any user-facing changes?
No
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
