lh0156 opened a new pull request, #23032: URL: https://github.com/apache/kafka/pull/23032
## Summary `RecordAppendResult` is created for every successful append. This change removes the incremental-strategy-only `outcome` and `extensionBytesNeeded` fields from regular results. Buffer-extension capacity is now held only by a private result subtype, while the existing append/new-batch behavior remains unchanged. The chunked accumulator consumes the capacity through a package-private accessor, and a regression test covers the buffer-extension result contract. ## Testing The implementation followed a TDD cycle: 1. RED: the new regression test failed to compile because `extensionBytesNeeded()` did not exist. 2. GREEN: implemented the result specialization and reran the test. Validated with: - `./gradlew :clients:test --tests org.apache.kafka.clients.producer.internals.RecordAccumulatorTest --tests org.apache.kafka.clients.producer.internals.ChunkedRecordAccumulatorTest --no-build-cache --console=plain` - `./gradlew :clients:test --no-build-cache --console=plain` - `./gradlew :clients:spotlessJavaCheck --no-build-cache --console=plain` The clients test task also ran checkstyle and SpotBugs successfully. -- 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]
