Skip to content

Move .Kill() func from inspector to go/mysql#1148

Merged
timvaillancourt merged 3 commits intogithub:masterfrom
timvaillancourt:mysql-utils-kill-func
Jul 11, 2022
Merged

Move .Kill() func from inspector to go/mysql#1148
timvaillancourt merged 3 commits intogithub:masterfrom
timvaillancourt:mysql-utils-kill-func

Conversation

@timvaillancourt
Copy link
Copy Markdown
Collaborator

@timvaillancourt timvaillancourt commented Jul 9, 2022

Description

This PR moves the new .Kill() function from #846 to go/mysql and removes the todo comment about finding a permanent spot for it

cc @ajm188

In case this PR introduced Go code changes:

  • contributed code is using same conventions as original code
  • script/cibuild returns with no formatting errors, build errors or unit test errors.

@timvaillancourt timvaillancourt requested review from a user and rashiq July 9, 2022 13:57
Copy link
Copy Markdown
Contributor

@ajm188 ajm188 left a comment

Choose a reason for hiding this comment

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

lgtm

@timvaillancourt timvaillancourt merged commit f527d63 into github:master Jul 11, 2022
@timvaillancourt timvaillancourt deleted the mysql-utils-kill-func branch July 11, 2022 09:02
RainbowDashy pushed a commit to RainbowDashy/gh-ost that referenced this pull request Jul 13, 2022
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