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

Remove legacy collator code and remove icu dependency #8241

Closed
windtalker opened this issue Oct 25, 2023 · 0 comments · Fixed by #8242
Closed

Remove legacy collator code and remove icu dependency #8241

windtalker opened this issue Oct 25, 2023 · 0 comments · Fixed by #8242
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@windtalker
Copy link
Contributor

Enhancement

Clickhouse's collator support depends on ICU lib, while TiFlash implement its own support for collator that is compactible with TiDB/MySQL, and it is not depend on ICU lib. This issue aims at removing the legacy Clickhouse's collator releated code and removing the dependency of ICU lib
@windtalker windtalker added the type/enhancement The issue or PR belongs to an enhancement. label Oct 25, 2023
@windtalker windtalker mentioned this issue Oct 25, 2023
Merged
12 tasks
ti-chi-bot bot pushed a commit that referenced this issue Oct 25, 2023
@windtalker
Remove legacy collator related code (#8242)
84ba14b 
close #8241
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove legacy collator related code windtalker/tiflash
1 participant
@windtalker