Skip to content

stats: do not wait for data unchanged when auto analyze (#7022) - #7093

Merged
coocood merged 3 commits into
pingcap:release-2.0from
alivxxx:auto-analyze
Jul 19, 2018
Merged

stats: do not wait for data unchanged when auto analyze (#7022)#7093
coocood merged 3 commits into
pingcap:release-2.0from
alivxxx:auto-analyze

Conversation

@alivxxx

@alivxxx alivxxx commented Jul 18, 2018

Copy link
Copy Markdown
Contributor

What have you changed? (mandatory)

Cherry pick #7022

What is the type of the changes? (mandatory)

  • Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)

Unit test

Does this PR affect documentation (docs/docs-cn) update? (mandatory)

No

Does this PR affect tidb-ansible update? (mandatory)

No

Does this PR need to be added to the release notes? (mandatory)

No

Refer to a related PR or issue link (optional)

Benchmark result if necessary (optional)

Add a few positive/negative examples (optional)

@shenli

shenli commented Jul 18, 2018

Copy link
Copy Markdown
Member

LGTM

@shenli shenli added the status/LGT1 Indicates that a PR has LGTM 1. label Jul 18, 2018

@zz-jason zz-jason left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@zz-jason zz-jason added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Jul 19, 2018
@alivxxx

alivxxx commented Jul 19, 2018

Copy link
Copy Markdown
Contributor Author

/run-all-tests tidb-test=release-2.0 tikv=release-2.0 pd=release-2.0

@coocood
coocood merged commit f7b7f1a into pingcap:release-2.0 Jul 19, 2018
@alivxxx
alivxxx deleted the auto-analyze branch July 19, 2018 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/statistics status/LGT2 Indicates that a PR has LGTM 2.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants