Skip to content
This repository has been archived by the owner on Aug 21, 2023. It is now read-only.

sql: fix table sample sql generator bug #262

Merged
merged 9 commits into from
Apr 7, 2021

Conversation

lichunzhu
Copy link
Contributor

What problem does this PR solve?

#260 introduced some bugs.

What is changed and how it works?

Fix them.

Check List

Tests

  • Unit test
  • Integration test

Release note

  • No release note

Copy link
Collaborator

@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.

can you add a test case showing what is fixed by this?

@lichunzhu
Copy link
Contributor Author

lichunzhu commented Apr 1, 2021

can you add a test case showing what is fixed by this?

@kennytm Already addressed, PTAL again.

@kennytm
Copy link
Collaborator

kennytm commented Apr 7, 2021

/lgtm

@ti-chi-bot
Copy link
Member

[REVIEW NOTIFICATION]

This pull request has been approved by:

  • kennytm

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Reviewer can indicate their review by writing /lgtm in a comment.
Reviewer can cancel approval by writing /lgtm cancel in a comment.

@ti-chi-bot ti-chi-bot added the status/LGT1 One reviewer approved (LGTM1) label Apr 7, 2021
@lichunzhu
Copy link
Contributor Author

/merge

@ti-chi-bot
Copy link
Member

This pull request has been accepted and is ready to merge.

Commit hash: 563fb64

@ti-chi-bot ti-chi-bot merged commit 51b465b into pingcap:master Apr 7, 2021
@lichunzhu lichunzhu deleted the fixMasterSampleBug branch April 7, 2021 06:42
lichunzhu added a commit to lichunzhu/dumpling that referenced this pull request Apr 14, 2021
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
tisonkun pushed a commit to tisonkun/tidb that referenced this pull request Oct 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/L status/can-merge status/LGT1 One reviewer approved (LGTM1)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants