Skip to content

MINOR: Fix flaky test shouldQuerySpecificActivePartitionStores#9873

Merged
guozhangwang merged 1 commit intoapache:trunkfrom
cadonna:KS4867-fix_integration_test
Jan 13, 2021
Merged

MINOR: Fix flaky test shouldQuerySpecificActivePartitionStores#9873
guozhangwang merged 1 commit intoapache:trunkfrom
cadonna:KS4867-fix_integration_test

Conversation

@cadonna
Copy link
Copy Markdown
Member

@cadonna cadonna commented Jan 12, 2021

Committer Checklist (excluded from commit message)

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

@cadonna
Copy link
Copy Markdown
Member Author

cadonna commented Jan 12, 2021

Call for review: @vvcephei

Copy link
Copy Markdown
Contributor

@guozhangwang guozhangwang left a comment

Choose a reason for hiding this comment

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

Did a quick check. LGTM.

@guozhangwang guozhangwang merged commit ee5ef89 into apache:trunk Jan 13, 2021
@guozhangwang
Copy link
Copy Markdown
Contributor

@cadonna LMK if we can cherry-pick this one to 2.7 immediately or should we wait for#9681

ijuma added a commit to ijuma/kafka that referenced this pull request Jan 13, 2021
* apache-github/trunk:
  Only schedule AlterIsr thread when we have an ISR change (apache#9749)
  MINOR: Fix flaky test shouldQuerySpecificActivePartitionStores (apache#9873)
  MINOR: Add restoration time tracking (apache#9830)
  MINOR: Remove unnecessary assertDoesNotThrow (apache#9854)
@cadonna
Copy link
Copy Markdown
Member Author

cadonna commented Jan 13, 2021

@guozhangwang Do you want me to open a PR to cherry-pick #9681 to 2.7 or do you want you cherry-pick it directly to 2.7?

guozhangwang pushed a commit that referenced this pull request Jan 15, 2021
@guozhangwang
Copy link
Copy Markdown
Contributor

Cherry-picked to 2.7

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