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.

Sign up for GitHub

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

Jump to bottom

Unify the calculation of defaultCF - block-cache-size #4086

Merged
merged 1 commit into from
Jul 29, 2020
Merged

Unify the calculation of defaultCF - block-cache-size #4086

merged 1 commit into from
Jul 29, 2020

Conversation

TomShawn
Copy link
Contributor

@TomShawn TomShawn commented Jul 29, 2020 •
edited
Loading

What is changed, added or deleted? (Required)

This PR cherry-picks the change made in #3420.

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):

@TomShawn TomShawn added type/enhancement The issue or PR belongs to an enhancement. translation/doing This PR’s assignee is translating this PR. size/small Changes of a small size. labels Jul 29, 2020
@glkappe glkappe mentioned this pull request Jul 29, 2020
Merged
5 tasks
@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Jul 29, 2020
@TomShawn TomShawn added require-LGT1 Indicates that the PR requires an LGTM. status/can-merge Indicates a PR has been approved by a committer. labels Jul 29, 2020
@ti-srebot
Copy link
Contributor

/run-all-tests

@ti-srebot ti-srebot merged commit 70389f5 into master Jul 29, 2020
@Joyinqin Joyinqin mentioned this pull request Jul 30, 2020
Merged
9 tasks
@Joyinqin
Copy link
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 Jul 30, 2020
@Joyinqin
Copy link
Contributor

/unlabel translation/doing

@ti-srebot ti-srebot removed the translation/doing This PR’s assignee is translating this PR. label Jul 30, 2020
@ti-srebot ti-srebot mentioned this pull request Jul 30, 2020
Merged
9 tasks
@lilin90 lilin90 deleted the TomShawn-patch-1 branch August 5, 2020 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@toutdesuite toutdesuite toutdesuite approved these changes

Assignees

@Joyinqin Joyinqin

Labels
require-LGT1 Indicates that the PR requires an LGTM. size/small Changes of a small size. status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@TomShawn @ti-srebot @Joyinqin @toutdesuite