Skip to content

system-variables: update noop functions#3898

Merged
TomShawn merged 5 commits intomasterfrom
unknown repository
Sep 18, 2020
Merged

system-variables: update noop functions#3898
TomShawn merged 5 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Sep 14, 2020

What is changed, added or deleted? (Required)

This is the PR for pingcap/tidb#19506

It can be reviewed now, but should merge after the TiDB PR.

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v4.0 (TiDB 4.0 versions)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

What is the related PR or file link(s)?

  • This PR is translated from:
  • Other reference link(s):

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Have version specific changes
  • Might cause conflicts

@ghost ghost added the status/PTAL This PR is ready for reviewing. label Sep 14, 2020
@TomShawn TomShawn added needs-cherry-pick-4.0 size/small Changes of a small size. translation/doing This PR's assignee is translating this PR. labels Sep 14, 2020
Copy link
Copy Markdown
Contributor

@jackysp jackysp left a comment

Choose a reason for hiding this comment

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

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Sep 14, 2020
@TomShawn
Copy link
Copy Markdown
Contributor

@nullnotnil Should release-4.0 be updated as well?

@ghost
Copy link
Copy Markdown
Author

ghost commented Sep 14, 2020

@nullnotnil Should release-4.0 be updated as well?

I have not tagged the TiDB PR for cherry-pick. But I am open to it if users request it, and will update docs accordingly.

Copy link
Copy Markdown
Contributor

@TomShawn TomShawn left a comment

Choose a reason for hiding this comment

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

Rest LGTM

Comment thread system-variables.md Outdated
@ti-srebot ti-srebot added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Sep 14, 2020
Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
@ti-srebot
Copy link
Copy Markdown
Contributor

@jackysp, @TomShawn, @ti-srebot, PTAL.

1 similar comment
@ti-srebot
Copy link
Copy Markdown
Contributor

@jackysp, @TomShawn, @ti-srebot, PTAL.

@jackysp
Copy link
Copy Markdown
Contributor

jackysp commented Sep 18, 2020

@TomShawn could you remove the label 'DNM'?

@TomShawn
Copy link
Copy Markdown
Contributor

@TomShawn could you remove the label 'DNM'?

Done. Should this PR be merged after pingcap/tidb#19506?

@jackysp
Copy link
Copy Markdown
Contributor

jackysp commented Sep 18, 2020

@TomShawn could you remove the label 'DNM'?

Done. Should this PR be merged after pingcap/tidb#19506?

Finally, it's merged.

Comment thread system-variables.md Outdated
@TomShawn TomShawn merged commit 5965a53 into pingcap:master Sep 18, 2020
@ghost ghost deleted the update-noop-functions branch September 20, 2020 22:19
@Joyinqin
Copy link
Copy Markdown
Contributor

/label translation/done

@ti-srebot ti-srebot added the translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. label Sep 24, 2020
@Joyinqin
Copy link
Copy Markdown
Contributor

/unlabel translation/doing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/small Changes of a small size. status/LGT2 Indicates that a PR has LGTM 2. status/PTAL This PR is ready for reviewing. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants