Skip to content

Increase segment.bytes configuration to 1MB in compaction and fetch a…#5352

Open
Ankith L (Ankith-Confluent) wants to merge 37 commits intodev_kip-932_queues-for-kafkafrom
dev_kip-932_ci_integration_tests_fixes
Open

Increase segment.bytes configuration to 1MB in compaction and fetch a…#5352
Ankith L (Ankith-Confluent) wants to merge 37 commits intodev_kip-932_queues-for-kafkafrom
dev_kip-932_ci_integration_tests_fixes

Conversation

@Ankith-Confluent
Copy link
Member

@Ankith-Confluent Ankith L (Ankith-Confluent) commented Mar 9, 2026

Fix mock broker share session management: add per-broker session lookup by NodeId, correct epoch mismatch handling to return INVALID_SHARE_SESSION_EPOCH, and add proper ShareFetch/ShareAcknowledge validation (epoch-0 ack rejection, epoch=-1 topic add/forget rejection). Update Kafka version to 4.2.0, add CI pipeline for share consumer integration tests (0170/0171), fix compaction test segment.bytes for Kafka 4.1+, and mark mock-only share tests as manual to avoid thread leak in local quick suite.

@confluent-cla-assistant
Copy link

🎉 All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

@airlock-confluentinc airlock-confluentinc bot force-pushed the dev_kip-932_ci_integration_tests_fixes branch 2 times, most recently from 013971b to 2c262c6 Compare March 12, 2026 10:23
This change updates several test functions to use SUB_TEST instead of
SUB_TEST_QUICK, ensuring consistency in test execution. The modifications
affect tests related to implicit acknowledgments, session expiry, and
error handling, among others.

Additionally, a comment is added in test.c to highlight the need for a
proper public API for accessing internal structures, which currently
leads to dependencies on internal struct layouts.
@airlock-confluentinc airlock-confluentinc bot force-pushed the dev_kip-932_ci_integration_tests_fixes branch from a777d8a to 7a23d48 Compare March 13, 2026 16:17
@Ankith-Confluent Ankith L (Ankith-Confluent) marked this pull request as ready for review March 13, 2026 17:14
@Ankith-Confluent Ankith L (Ankith-Confluent) requested a review from a team as a code owner March 13, 2026 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant