Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

stats: disable stats by default (#693) #697

Conversation

ti-srebot
Copy link
Contributor

@ti-srebot ti-srebot commented Jan 14, 2021

cherry-pick #693 to release-5.0-rc
You can switch your code base to this Pull Request by using git-extras:

# In br repo:
git pr https://github.com/pingcap/br/pull/697

After apply modifications, you can push your change to this PR via:

git push git@github.com:ti-srebot/br.git pr/697:release-5.0-rc-87eb3ed13336

What problem does this PR solve?

Disable backup stats by default. because of

  1. DumpStatsToJson is not stable
  2. It increases memory usage may cause BR OOM

What is changed and how it works?

  1. set cfg.IgnoreStats default to true
  2. if cfg.IgnoreStats is true, set statsLease to -1

Check List

Tests

  • Integration test

Related changes

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

Release Note

  • Disable backup stats by default to avoid BR reached OOM issue during backup.

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot added Priority/P0 Top priority issue. Must have an associated milestone release-blocker type/5.0-rc-cherry-pick labels Jan 14, 2021
@3pointer 3pointer closed this Jan 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Priority/P0 Top priority issue. Must have an associated milestone release-blocker type/5.0-rc-cherry-pick
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants