8000 Snowball 2: Update supported languages by Simran-B · Pull Request #374 · arangodb/docs · GitHub
[go: up one dir, main page]
Skip to content
This repository was archived by the owner on Dec 13, 2023. It is now read-only.

Snowball 2: Update supported languages #374

Merged
merged 7 commits into from
Mar 19, 2020
Merged

Snowball 2: Update supported languages #374

merged 7 commits into from
Mar 19, 2020

Conversation

Simran-B
Copy link
Collaborator
@Simran-B Simran-B commented Mar 18, 2020 •
edited
Loading

I added a hint regarding lexicographical sorting, but I think it's quite right.

FOR doc IN coll SORT doc.letter RETURN doc sorts according to --default-language, but so does
FOR doc IN view SORT doc.letter RETURN doc.

What seems to be affected are range queries, e.g.
FILTER doc.letter >= "X" AND doc.letter <= "Þ" SORT doc.letter. This returns the expected range of characters for Icelandic using a collection, but not using a View:
SEARCH ANALYZER(doc.letter >= "X" AND doc.letter <= "Þ", "text_is") SORT doc.letter

Andrey confirmed that SEARCH does not honor the Analyzer locale nor the defined server language. It's unclear whether the behavior is undefined or comparing characters codepoints.

We should probably add this to the Known Issues list!

Also see arangodb/arangodb#11046 (comment)

@Simran-B Simran-B added this to the 3.7 milestone Mar 18, 2020
@Simran-B Simran-B requested a review from gnusi March 18, 2020 15:03
@Simran-B Simran-B self-assigned this Mar 18, 2020
@Simran-B Simran-B marked this pull request as ready for review March 18, 2020 15:22
@Simran-B Simran-B mentioned this pull request Mar 18, 2020
Merged
gnusi
gnusi approved these changes Mar 19, 2020
View reviewed changes
Copy link
Contributor
@gnusi gnusi left a comment

Choose a reason for hiding this comment

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

I think that'd be better to attach this notice to the description of supported SEARCH functions.
IN_RANGE, >, >=, <, <=, STARTS_WITH are affected

@Simran-B Simran-B mentioned this pull request Mar 19, 2020
Closed
@Simran-B Simran-B merged commit e1d9567 into master Mar 19, 2020
@Simran-B Simran-B deleted the snowball2 branch March 19, 2020 16:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@gnusi gnusi gnusi approved these changes

Assignees

@Simran-B Simran-B

Labels
1 Feature
Projects
None yet
Milestone
3.7
Development

Successfully merging this pull request may close these issues.
2 participants
@Simran-B @gnusi
0