Skip to content
This repository has been archived by the owner on Aug 21, 2023. It is now read-only.

export: delete default tidb mem quota query configuration #233

Merged
merged 4 commits into from
Jan 13, 2021

Conversation

lichunzhu
Copy link
Contributor

What problem does this PR solve?

#232
dumpling's default tidb-mem-quota-query is 32GB which may cause TiDB OOM.

What is changed and how it works?

Don't set tidb-mem-quota-query when it's not specified (Use TiDB's default configuration).

Check List

Tests

  • Unit test
  • Integration test

Side effects

  • Possible performance regression

Related changes

  • Need to cherry-pick to the release branch
  • Need to update the documentation

Release note

  • Don't set tidb_mem_quota_query when it's not specified.

@glorv
Copy link
Collaborator

glorv commented Jan 12, 2021

LGTM

lance6716
lance6716 previously approved these changes Jan 12, 2021
Copy link
Collaborator

@lance6716 lance6716 left a comment

Choose a reason for hiding this comment

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

LGTM

@overvenus
Copy link
Member

How to test your change?

kennytm
kennytm previously approved these changes Jan 13, 2021
Copy link
Collaborator

@kennytm kennytm left a comment

Choose a reason for hiding this comment

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

lgtm

@lichunzhu lichunzhu dismissed stale reviews from kennytm and lance6716 via 8914648 January 13, 2021 03:58
@lichunzhu
Copy link
Contributor Author

How to test your change?

I add an integration test in this PR. PTAL

@overvenus
Copy link
Member

LGTM

@lichunzhu lichunzhu merged commit ddc6354 into pingcap:master Jan 13, 2021
@lichunzhu lichunzhu deleted the refactorTiDBMemQuota branch January 13, 2021 07:10
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
* delete default tidb mem quota query configuration
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
* delete default tidb mem quota query configuration
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
* delete default tidb mem quota query configuration
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
* delete default tidb mem quota query configuration
tisonkun pushed a commit to tisonkun/tidb that referenced this pull request Oct 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants