Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

infoschema: remove redundant test introduce by （#11417) #11422

Merged
merged 2 commits into from
Jul 25, 2019
Merged

infoschema: remove redundant test introduce by （#11417) #11422

merged 2 commits into from
Jul 25, 2019

Conversation

crazycs520
Copy link
Contributor

What problem does this PR solve?

remove redundant test introduce by （#11417)
Sorry for my mistake.

What is changed and how it works?

Check List

Tests

  • No code

@sre-bot
Copy link
Contributor

sre-bot commented Jul 24, 2019

Thanks for your PR.
This PR will be closed by bot because you already had 3 opened PRs, close or merge them before submitting a new one.
#10673 , #11419 , #11421

@sre-bot sre-bot closed this Jul 24, 2019
@codecov
Copy link

codecov bot commented Jul 24, 2019 •
edited
Loading

Codecov Report

Merging #11422 into master will decrease coverage by 0.1803%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             master     #11422        +/-   ##
================================================
- Coverage   81.5543%   81.3739%   -0.1804%     
================================================
  Files           424        424                
  Lines         91398      90733       -665     
================================================
- Hits          74539      73833       -706     
- Misses        11568      11598        +30     
- Partials       5291       5302        +11

@crazycs520 crazycs520 reopened this Jul 25, 2019
@crazycs520
Copy link
Contributor Author

/run-all-tests

@crazycs520
Copy link
Contributor Author

@winkyao @zz-jason PTAL

zz-jason
zz-jason reviewed Jul 25, 2019
View reviewed changes
Copy link
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

@zz-jason zz-jason added status/LGT1 Indicates that a PR has LGTM 1. type/enhancement The issue or PR belongs to an enhancement. component/test labels Jul 25, 2019
winkyao
winkyao reviewed Jul 25, 2019
View reviewed changes
Copy link
Contributor

@winkyao winkyao left a comment

Choose a reason for hiding this comment

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

LGTM

@winkyao winkyao merged commit 9e9a564 into pingcap:master Jul 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zz-jason zz-jason zz-jason left review comments

@winkyao winkyao winkyao approved these changes

Assignees
No one assigned
Labels
component/test status/LGT1 Indicates that a PR has LGTM 1. type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@crazycs520 @sre-bot @zz-jason @winkyao