-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Clickhouse string ordering and string filtering by UTF8 instead of bytes #6143
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
Open
casab
wants to merge
42
commits into
cube-js:master
Choose a base branch
from
casab:feature/clickhouse_utf8_filter_order
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 35 commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
63b935c
Remove unnecessary concat function, and use UTF8 compatible functions…
casab 1ea8d11
Merge remote-tracking branch 'upstream/master' into feature/clickhous…
casab 49111c2
Add integration testing for ordering with collation
casab 153b3fd
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab b2c4ee9
Move case sensitive entry from ClickHouseDbRunner.js to ClickHouseDbR…
casab dd2fd32
Added back CONCAT() to prevent sql injection
casab f26f4fe
Merge branch 'master' into feature/clickhouse_utf8_filter_order
KSDaemon a5b279e
Change intentation from 4 spaces to 2 spaces
casab f7a4759
Only apply coalition if field's type is string
casab 83437e9
Merge branch 'master' into feature/clickhouse_utf8_filter_order
KSDaemon 600c1fa
Replaced Google with Gork to make ordering clear, added toValidUTF8 t…
casab eadfa1a
Added CUBEJS_DB_CLICKHOUSE_SORT_COLLATION env variable for collation …
casab 0610d16
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab 8807178
Only use collation for clickhouse if env variable is set
casab f095335
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab e76b91a
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab fdebe6e
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab be6875b
Add CUBEJS_DB_CLICKHOUSE_USE_COLLATION env var to disable collation, …
casab 4fc9ed4
Fix default clickhouseSortCollection value
casab b44e57e
Use ILIKE instead of LIKE with lowerUTF8 and toValidUTF8
casab a2616fe
Merge branch 'master' into feature/clickhouse_utf8_filter_order
KSDaemon ce3ec56
Fix clickhouseUseCollation tests to check for default value
casab ee06aee
Merge branch 'master' into feature/clickhouse_utf8_filter_order
KSDaemon b649df2
Fix clickhouse integration tests for the missing values
casab 716f24b
Merge branch 'master' into feature/clickhouse_utf8_filter_order
KSDaemon 79ab001
Assert collation in the 'collation in order by' clickhouse integratio…
casab f660674
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab beac449
Fix getFieldType by using hash.id instead
casab bd29d16
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab a62eb42
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab 225c19a
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab 8971b45
Merge branch 'master' into feature/clickhouse_utf8_filter_order
KSDaemon f712829
Merge branch 'master' into feature/clickhouse_utf8_filter_order
KSDaemon d18b953
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab 79373d6
Merge branch 'master' into feature/clickhouse_utf8_filter_order
KSDaemon 8179923
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab 7a809cd
Make getField and getCollation private
casab 97aeae4
Use plain js instead of ramda for collation check
casab 2379931
Move templates/select, templates/expressions/sort, templates/expressi…
casab 8d61d13
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab 8672872
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab ca0612c
Merge branch 'master' into feature/clickhouse_utf8_filter_order
casab File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.