Skip to content

Conversation

@UladzislauBlok
Copy link
Contributor

@UladzislauBlok UladzislauBlok commented Nov 15, 2025

Fix for flaky testConcurrentRemoveReadForCache1

Short description of root cause:

Before scheduler starts clean up, he's triggering read thread. If it's
happening before RemoteIndexCache#remove does remove entity from cache,
reader thread will successfully read entity, and won't trigger fetching
from remote log

Full analysis:

https://issues.apache.org/jira/browse/KAFKA-19286

Evidence:

Run entire test class and annotate testConcurrentRemoveReadForCache1 as
RepeatedTest to run 100_000 times

Result:

./gradlew storage:test --tests org.apache.kafka.storage.internals.log.RemoteIndexCacheTest ...
BUILD SUCCESSFUL in 48m 11s

@github-actions github-actions bot added triage PRs from the community tests Test fixes (including flaky tests) storage Pull requests that target the storage module small Small PRs labels Nov 15, 2025
@parthikjadav
Copy link

Hi everyone! 👋

I’ve proposed a new feature that could really improve the project, and I’d love some help bringing it to life.
If anyone is interested in contributing, here’s the issue with full details:

👉 https://github.com/parthikjadav/atron-js/issues/1#event-20971767962

Your input, ideas, or PRs would be super appreciated!
Thanks in advance to anyone who wants to collaborate. 🚀

@github-actions
Copy link

A label of 'needs-attention' was automatically added to this PR in order to raise the
attention of the committers. Once this issue has been triaged, the triage label
should be removed to prevent this automation from happening again.

@UladzislauBlok
Copy link
Contributor Author

@chia7712 @mimaison Hello. This is fix for flaky test testConcurrentRemoveReadForCache1. Could you please take a look?

Copy link
Member

@mimaison mimaison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR

@mimaison mimaison merged commit c7bdabf into apache:trunk Dec 4, 2025
22 checks passed
@github-actions github-actions bot removed the triage PRs from the community label Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

small Small PRs storage Pull requests that target the storage module tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants