showuon opened a new pull request, #14373: URL: https://github.com/apache/kafka/pull/14373
waiting for https://github.com/apache/kafka/pull/14347 . == This PR did: 1. enable `testFencingOnTransactionExpiration` 2. Verify the log start offset and local log start offset before and after committing/aborting txn, to make sure we will move the segment to remote storage after last stable offset advanced. For (1), the reason the test failed before because: 1. RLMM is not ready since `remote.log.metadata.initialization.retry.interval.ms` is too long to wait for retrying initialization. Reducing it to 200 ms to fix the issue. 2. Even though we extends the `TransactionsTest`, we didn't inherit some changes in the class initialization. And that causes the test doesn't run as we expected, ex: `TransactionsTopicPartitionsProp` is using 50 in tiered storage case, but it should be 3... etc. Fixed it by inherit all configs in parent class now. ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation - [ ] Verify test coverage and CI build status - [ ] Verify documentation (including upgrade notes) -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
