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

fix pump init order, register the node after listen ports (#709) #716

Merged
merged 1 commit into from
Aug 16, 2019

Conversation

zier-one
Copy link
Contributor

@zier-one zier-one commented Aug 15, 2019

What problem does this PR solve?

cherry-pick #709 to release-2.1

@zier-one zier-one requested a review from july2993 August 15, 2019 13:11
@july2993
Copy link
Contributor

/run-all-tests

Copy link
Contributor

@july2993 july2993 left a comment

Choose a reason for hiding this comment

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

LGTM

@zier-one
Copy link
Contributor Author

@suzaku PTAL

Copy link
Contributor

@suzaku suzaku left a comment

Choose a reason for hiding this comment

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

LGTM

@july2993 july2993 merged commit 2996270 into pingcap:release-2.1 Aug 16, 2019
@zier-one zier-one deleted the cp711 branch August 20, 2019 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants