Skip to content

cmd: migrate the Execute interface to ExecuteInternal in cmd package (#22647)#22661

Closed
ti-srebot wants to merge 1 commit intopingcap:release-5.0-rcfrom
ti-srebot:release-5.0-rc-d57231205fb3
Closed

cmd: migrate the Execute interface to ExecuteInternal in cmd package (#22647)#22661
ti-srebot wants to merge 1 commit intopingcap:release-5.0-rcfrom
ti-srebot:release-5.0-rc-d57231205fb3

Conversation

@ti-srebot
Copy link
Contributor

@ti-srebot ti-srebot commented Feb 2, 2021

cherry-pick #22647 to release-5.0-rc
You can switch your code base to this Pull Request by using git-extras:

# In tidb repo:
git pr https://github.com/pingcap/tidb/pull/22661

After apply modifications, you can push your change to this PR via:

git push git@github.com:ti-srebot/tidb.git pr/22661:release-5.0-rc-d57231205fb3

Signed-off-by: AilinKid 314806019@qq.com

What problem does this PR solve?

Issue Number: part close #https://github.com/pingcap/tidb-test/issues/1152

What is changed and how it works?

What's Changed:

Make the code in cmd package get ride of original Execute API which may have some security risks.

Related changes

  • PR to update pingcap/docs/pingcap/docs-cn:
  • Need to cherry-pick to the release branch

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
the bench test can only be tested in the real cluster.
1: start a cluster
2: run the bench test main.

Release note

  • cmd: migrate the Execute interface to ExecuteInternal in cmd package.

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot
Copy link
Contributor Author

/run-all-tests

@ti-srebot
Copy link
Contributor Author

@AilinKid you're already a collaborator in bot's repo.

@bb7133
Copy link
Member

bb7133 commented Feb 4, 2021

We don't need this PR, close it.

@bb7133 bb7133 closed this Feb 4, 2021
@tiancaiamao tiancaiamao deleted the release-5.0-rc-d57231205fb3 branch February 4, 2021 06:32
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