Skip to content

MINOR: StopReplicaResp and StopReplicaReq Test should cover all available version#10068

Merged
chia7712 merged 3 commits into
apache:trunkfrom
dengziming:minor-stop-replica-latest-version
Feb 7, 2021
Merged

MINOR: StopReplicaResp and StopReplicaReq Test should cover all available version#10068
chia7712 merged 3 commits into
apache:trunkfrom
dengziming:minor-stop-replica-latest-version

Conversation

@dengziming

@dengziming dengziming commented Feb 5, 2021

Copy link
Copy Markdown
Member

More detailed description of your change
Accidently found that the test case in StopReplicaRespTest and StopReplicaReqTest didn't cover all versions.

Summary of testing strategy (including rationale)
QA

Committer Checklist (excluded from commit message)

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

@dengziming dengziming changed the title MINOR: MINOR: StopReplicaResp and StopReplicaReq Test should cover all available version MINOR: StopReplicaResp and StopReplicaReq Test should cover all available version Feb 5, 2021
dengziming added 2 commits February 5, 2021 19:23

@dajac dajac left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good catch! LGTM.

@dajac

dajac commented Feb 5, 2021

Copy link
Copy Markdown
Member

I've re-triggered Jenkins. I will merge once we have a clean run.

@dengziming

Copy link
Copy Markdown
Member Author

Failures seem unrelated to this pr.

@chia7712

chia7712 commented Feb 7, 2021

Copy link
Copy Markdown
Member

@dengziming Nice patch!

@chia7712
chia7712 merged commit 4cab609 into apache:trunk Feb 7, 2021
@dengziming
dengziming deleted the minor-stop-replica-latest-version branch August 28, 2021 13:58
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.

3 participants