Skip to content

fix: handle empty read-committed elastic log fetch#3346

Merged
1sonofqiu merged 1 commit into
mainfrom
fix/elastic-log-empty-aborted-txns
May 8, 2026
Merged

fix: handle empty read-committed elastic log fetch#3346
1sonofqiu merged 1 commit into
mainfrom
fix/elastic-log-empty-aborted-txns

Conversation

@1sonofqiu
Copy link
Copy Markdown
Contributor

Summary

  • return an empty aborted transaction list for empty ElasticLog fetch results under read-committed isolation
  • avoid calling ElasticLogSegment.fetchUpperBoundOffset for zero-byte fetch results
  • add a regression test for maxLength=0 with includeAbortedTxns=true

Tests

  • ./gradlew core:test --tests kafka.log.streamaspect.ElasticLogTest --no-daemon -x checkstyleTest

Notes

  • Running the same targeted test without -x checkstyleTest was blocked by an existing untracked file in the working tree: core/src/test/java/kafka/coordinator/group/GroupCoordinatorPartitionTest.java violates MethodLength. That file is not part of this PR.

@1sonofqiu 1sonofqiu force-pushed the fix/elastic-log-empty-aborted-txns branch from 4b3184a to f27dd4c Compare May 8, 2026 03:13
@1sonofqiu 1sonofqiu merged commit 9be17ab into main May 8, 2026
6 checks passed
@1sonofqiu 1sonofqiu deleted the fix/elastic-log-empty-aborted-txns branch May 8, 2026 03:40
1sonofqiu added a commit that referenced this pull request May 8, 2026
1sonofqiu added a commit that referenced this pull request May 8, 2026
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.

2 participants