Skip to content

Conversation

@ran-huang
Copy link
Contributor

What is changed, added or deleted? (Required)

Change blacklist to blocklist, whitelist to allowlist.

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)

If you select two or more versions from above, to trigger the bot to cherry-pick this PR to your desired release version branch(es), you must add corresponding labels such as needs-cherry-pick-4.0, needs-cherry-pick-3.1, needs-cherry-pick-3.0, and needs-cherry-pick-2.1.

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

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

@ran-huang ran-huang added requires-version-specific-changes After cherry-picked, the cherry-picked PR requires further changes. translation/doing This PR's assignee is translating this PR. needs-cherry-pick-2.1 labels Jun 11, 2020
@ran-huang ran-huang requested a review from lilin90 June 11, 2020 04:21
@ran-huang ran-huang self-assigned this Jun 11, 2020
Comment on lines 51 to 53
1. Insert the function or operator name and the collection of storage types to be prohibited from the function pushdown to `mysql.expr_pushdown_blacklist`.

2. Execute the `admin reload expr_pushdown_blacklist;` command.
Copy link
Member

Choose a reason for hiding this comment

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

It seems that we need to modify words in the two steps first, to keep consistency and avoid confusion.

@lilin90 lilin90 added the status/WIP This PR is still working in progress. label Jun 16, 2020
@ran-huang ran-huang added translation/no-need No need to translate this PR. and removed status/WIP This PR is still working in progress. translation/doing This PR's assignee is translating this PR. labels Jul 27, 2020
@ran-huang ran-huang requested a review from lilin90 July 27, 2020 07:09
@ran-huang
Copy link
Contributor Author

ran-huang commented Jul 27, 2020

@lilin90 I think this PR is ready for merge. PTAL, thanks!
Some components (Syncer, Binlog, TiSpark) will be edited in the next batch of updates.

Also, I removed the needs-cherry-pick-3.1/3.0/2.1 label because these branches might have severe conflicts. I'll update them manually.

Copy link
Member

@lilin90 lilin90 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 Jul 30, 2020
@lilin90 lilin90 added the require-LGT1 Indicates that the PR requires an LGTM. label Jul 30, 2020
@lilin90
Copy link
Member

lilin90 commented Jul 30, 2020

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Jul 30, 2020
@ti-srebot
Copy link
Contributor

/run-all-tests

@ti-srebot ti-srebot merged commit f242db8 into pingcap:master Jul 30, 2020
@ran-huang ran-huang deleted the replace-blacklist branch July 30, 2020 08:44
ti-srebot pushed a commit to ti-srebot/docs that referenced this pull request Jul 30, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot
Copy link
Contributor

cherry pick to release-4.0 in PR #3487

ti-srebot added a commit that referenced this pull request Jul 30, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ran-huang ran-huang mentioned this pull request Jul 30, 2020
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

require-LGT1 Indicates that the PR requires an LGTM. requires-version-specific-changes After cherry-picked, the cherry-picked PR requires further changes. status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1. translation/no-need No need to translate this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants