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.

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

test: fix integration test in pingcap/parser#808 #16589

Closed
wants to merge 1 commit into from

Conversation

tangenta
Copy link
Contributor

Fix the integration test in pingcap/parser#808

@tangenta tangenta requested a review from a team as a code owner April 20, 2020 03:37
@ghost ghost requested review from SunRunAway and removed request for a team April 20, 2020 03:37
@github-actions github-actions bot added sig/sql-infra SIG: SQL Infra sig/execution SIG execution labels Apr 20, 2020
@tangenta tangenta requested a review from kennytm April 20, 2020 03:57
@SunRunAway SunRunAway removed their request for review May 31, 2020 16:24
@AndreMouche
Copy link
Contributor

/LGTM

@ti-srebot
Copy link
Contributor

@AndreMouche, Thanks for your review. The bot only counts LGTMs from Reviewers and higher roles, but you're still welcome to leave your comments. See the corresponding SIG page for more information. Related SIGs: ddl(slack),execution(slack).

@sre-bot
Copy link
Contributor

sre-bot commented Oct 27, 2020

Copy link
Contributor

@kennytm kennytm left a comment

Choose a reason for hiding this comment

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

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Oct 27, 2020
@tangenta tangenta closed this Nov 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/executor component/parser sig/execution SIG execution sig/sql-infra SIG: SQL Infra status/LGT1 Indicates that a PR has LGTM 1.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants