Skip to content

KAFKA-20328 : Part 1 - headers-aware api test coverage#22196

Open
muralibasani wants to merge 4 commits intoapache:trunkfrom
muralibasani:KAFKA-20328-1
Open

KAFKA-20328 : Part 1 - headers-aware api test coverage#22196
muralibasani wants to merge 4 commits intoapache:trunkfrom
muralibasani:KAFKA-20328-1

Conversation

@muralibasani
Copy link
Copy Markdown
Contributor

@muralibasani muralibasani commented May 2, 2026

Ref : https://issues.apache.org/jira/browse/KAFKA-20328

After audit, found a few gaps and also considered the work of 1271 kip
This is first PR. Next PRs will be on Decorator/forwarder coverage — AbstractReadWriteDecorator and AbstractReadX related and other integration tests

This PR :

  • new tests for PlainToHeadersWindowStoreIteratorAdapterTest, RocksDBMigratingWindowStoreWithHeaders and TimestampedToHeadersStoreAdapter (considered deleted files in KAFKA-20224: Refactor Streams iterator adapters to reduce duplication #21830)
  • Update StoresTest with coverage for all 6 *WithHeaders factory methods on Stores
  • New test : DslStoreFormatTest cases for HEADERS enum
  • New contract tests for TimestampedKeyValueStoreWithHeaders, TimestampedWindowStoreWithHeaders and SessionStoreWithHeaders (with put, get, range, fetch, and backwardFetch, etc)

@github-actions github-actions Bot added triage PRs from the community streams tests Test fixes (including flaky tests) labels May 2, 2026
@muralibasani muralibasani changed the title KAFKA-20328 : Headers-aware api test coverage KAFKA-20328 : Part 1 - Headers-aware api test coverage May 2, 2026
@muralibasani muralibasani changed the title KAFKA-20328 : Part 1 - Headers-aware api test coverage KAFKA-20328 : Part 1 - headers-aware api test coverage May 2, 2026
@muralibasani muralibasani marked this pull request as ready for review May 2, 2026 20:29
@muralibasani
Copy link
Copy Markdown
Contributor Author

@aliehsaeedii pls take a look.
Considered the comments from the previous pr.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

streams tests Test fixes (including flaky tests) triage PRs from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant