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

server: support ComChangeUser #6638

Merged
merged 4 commits into from May 24, 2018
Merged

server: support ComChangeUser #6638

merged 4 commits into from May 24, 2018

Conversation

coocood
Copy link
Member

@coocood coocood commented May 24, 2018

cherry-pick #6623 from release-2.0 to master

* server: support ComChangeUser

* *: simplify logic

* address comment
@shenli
Copy link
Member

shenli commented May 24, 2018

LGTM

@shenli
Copy link
Member

shenli commented May 24, 2018

/run-all-tests

@shenli shenli added the status/LGT1 Indicates that a PR has LGTM 1. label May 24, 2018
@shenli
Copy link
Member

shenli commented May 24, 2018

/run-all-tests

@tiancaiamao
Copy link
Contributor

LGTM

@shenli shenli added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels May 24, 2018
@shenli shenli merged commit 2bcef0f into pingcap:master May 24, 2018
@coocood coocood deleted the change-user branch May 26, 2018 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/LGT2 Indicates that a PR has LGTM 2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants