Skip to content

stats: refine the row count estimation for outdated stats (#7175)#7185

Merged
coocood merged 1 commit into
pingcap:release-2.0from
alivxxx:estimate
Jul 30, 2018
Merged

stats: refine the row count estimation for outdated stats (#7175)#7185
coocood merged 1 commit into
pingcap:release-2.0from
alivxxx:estimate

Conversation

@alivxxx
Copy link
Copy Markdown
Contributor

@alivxxx alivxxx commented Jul 30, 2018

What have you changed? (mandatory)

Cherry pick #7175

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)

@alivxxx
Copy link
Copy Markdown
Contributor Author

alivxxx commented Jul 30, 2018

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

Copy link
Copy Markdown
Member

@zz-jason zz-jason left a comment

Choose a reason for hiding this comment

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

LGTM

@shenli shenli added the status/LGT1 Indicates that a PR has LGTM 1. label Jul 30, 2018
@shenli
Copy link
Copy Markdown
Member

shenli commented Jul 30, 2018

@coocood PTAL

Copy link
Copy Markdown
Member

@coocood coocood left a comment

Choose a reason for hiding this comment

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

LGTM

@coocood coocood merged commit 71074b3 into pingcap:release-2.0 Jul 30, 2018
@coocood coocood added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Jul 30, 2018
@alivxxx alivxxx deleted the estimate branch July 30, 2018 11:06
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