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

copr: Add mysql Enum and Set support #8849

Merged
merged 15 commits into from
Oct 28, 2020
Merged

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Oct 19, 2020

What problem does this PR solve?

Issue Number: tikv/rfcs#57

What is changed and how it works?

Proposal: chunk-based enum and set support in coprocessor

This PR will add mysql enum and set support.

Check List

Tests

  • Unit test

@skyzh
Copy link
Member

skyzh commented Oct 26, 2020

I would recommend split this into several PRs. You may first add enum and set types, then implement chunk vectors, and finally modify other parts in copr. This way, we could move on really fast.

@Xuanwo
Copy link
Member Author

Xuanwo commented Oct 26, 2020

OK

Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo changed the title copr: Add enum/set support copr: Add mysql Enum and Set support Oct 27, 2020
@Xuanwo Xuanwo marked this pull request as ready for review October 27, 2020 05:45
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo
Copy link
Member Author

Xuanwo commented Oct 27, 2020

ping @skyzh, please review again~

Signed-off-by: Xuanwo <github@xuanwo.io>
@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Oct 27, 2020
@skyzh
Copy link
Member

skyzh commented Oct 27, 2020

/run-all-tests

Copy link
Member

@breezewish breezewish left a comment

Choose a reason for hiding this comment

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

The rest LGTM

@skyzh
Copy link
Member

skyzh commented Oct 27, 2020

/run-all-tests

Signed-off-by: Xuanwo <github@xuanwo.io>
@ti-srebot ti-srebot removed the status/LGT1 Indicates that a PR has LGTM 1. label Oct 28, 2020
@ti-srebot ti-srebot added the status/LGT2 Indicates that a PR has LGTM 2. label Oct 28, 2020
@skyzh
Copy link
Member

skyzh commented Oct 28, 2020

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Oct 28, 2020
@ti-srebot
Copy link
Contributor

/run-all-tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants