arouel opened a new pull request, #3462: URL: https://github.com/apache/parquet-java/pull/3462
### Rationale for this change `InternalParquetRecordWriter.checkBlockSizeReached` runs in a hot path, and we observed unnecessary overhead from debug log argument handling when debug logging is disabled. At high throughput (millions of records), this avoidable work can accumulate and impact write efficiency. ### What changes are included in this PR? This PR adds `LOG.isDebugEnabled()` guards around debug log statements in `InternalParquetRecordWriter.checkBlockSizeReached`. The change is intentionally minimal and does not alter flush thresholds, row-group behavior, or any functional write logic. ### Are these changes tested? No new tests were added because the change is non-functional and limited to logging guards. Existing writer tests continue to cover the affected code paths functionally, and behavior is expected to remain unchanged. ### Are there any user-facing changes? There are no user-facing behavior changes. The expected impact is reduced internal overhead in high-volume write scenarios when debug logging is not enabled. Closes #3461 -- 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]
