Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MINOR: close mockClient within AbstractCoordinatorTest #18538

Closed
wants to merge 3 commits into from

Conversation

TaiJuWu
Copy link
Contributor

@TaiJuWu TaiJuWu commented Jan 14, 2025

When I try resolve flaky test AbstractCoordinatorTest, I found we don't close mockClient but these issues can't reproduce on my local so I can't verify it is useful or not.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@github-actions github-actions bot added triage PRs from the community core Kafka Broker consumer tests Test fixes (including flaky tests) clients small Small PRs labels Jan 14, 2025
@TaiJuWu TaiJuWu changed the title MINOR: close mockClient for AbstractCoordinatorTest MINOR: close mockClient within AbstractCoordinatorTest Jan 14, 2025
@ijuma
Copy link
Member

ijuma commented Jan 15, 2025

Can we merge master? It has a fix that will fix the build timeouts.

@TaiJuWu
Copy link
Contributor Author

TaiJuWu commented Jan 15, 2025

Can we merge master? It has a fix that will fix the build timeouts.

Sure.

@TaiJuWu
Copy link
Contributor Author

TaiJuWu commented Jan 15, 2025

Revert the flaky tag since it still happened in previous CI.

@TaiJuWu TaiJuWu closed this Jan 16, 2025
@TaiJuWu TaiJuWu deleted the close_coordinator branch January 16, 2025 01:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-approved clients consumer core Kafka Broker small Small PRs tests Test fixes (including flaky tests) triage PRs from the community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants