Skip to content

executor: move testShowStatsSuite into SerialSuites#15062

Merged
sre-bot merged 5 commits intopingcap:masterfrom
SunRunAway:issue14931-TestShowAnalyze
Mar 6, 2020
Merged

executor: move testShowStatsSuite into SerialSuites#15062
sre-bot merged 5 commits intopingcap:masterfrom
SunRunAway:issue14931-TestShowAnalyze

Conversation

@SunRunAway
Copy link
Contributor

What problem does this PR solve?

Fixes #14931

What is changed and how it works?

move testShowStatsSuite into SerialSuites

Check List

Tests

  • Unit test

@SunRunAway SunRunAway requested review from glorv and tiancaiamao March 2, 2020 07:25
@codecov
Copy link

codecov bot commented Mar 2, 2020

Codecov Report

Merging #15062 into master will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master     #15062   +/-   ##
===========================================
  Coverage   80.3219%   80.3219%           
===========================================
  Files           503        503           
  Lines        132391     132391           
===========================================
  Hits         106339     106339           
  Misses        17682      17682           
  Partials       8370       8370

@tiancaiamao
Copy link
Contributor

/run-all-tests

@tiancaiamao
Copy link
Contributor

LGTM

@tiancaiamao tiancaiamao added the status/LGT1 Indicates that a PR has LGTM 1. label Mar 3, 2020
@SunRunAway SunRunAway requested review from eurekaka and wshwsh12 March 3, 2020 08:17
@SunRunAway
Copy link
Contributor Author

PTAL~

@glorv
Copy link
Contributor

glorv commented Mar 5, 2020

LGTM

Copy link
Contributor

@wshwsh12 wshwsh12 left a comment

Choose a reason for hiding this comment

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

LGTM

@wshwsh12 wshwsh12 added status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. status/PTAL labels Mar 6, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Mar 6, 2020

/run-all-tests

@sre-bot sre-bot merged commit 1d28e7c into pingcap:master Mar 6, 2020
@SunRunAway SunRunAway deleted the issue14931-TestShowAnalyze branch March 6, 2020 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/test status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. type/bugfix This PR fixes a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

unstable unit test: testShowStatsSuite.TestShowAnalyzeStatus

5 participants