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

Make operators and functions in alphabetical order #4191

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ILuffZhe
Copy link
Contributor

No description provided.

@mihaibudiu
Copy link
Contributor

If you want this to stay this way you may need to add some kind of linter rule to enforce it.
Not sure how easy it is to parse reference.md to check it, though.

@ILuffZhe
Copy link
Contributor Author

If you want this to stay this way you may need to add some kind of linter rule to enforce it. Not sure how easy it is to parse reference.md to check it, though.

The CI failure is caused by CALCITE-6827, and it's OK now.

| s | STR_TO_MAP(string [, stringDelimiter [, keyValueDelimiter]]) | Returns a map after splitting the *string* into key/value pairs using delimiters. Default delimiters are ',' for *stringDelimiter* and ':' for *keyValueDelimiter*. Note that calcite is using the LAST_WIN strategy
| s | SUBSTRING_INDEX(string, delim, count) | Returns the substring from *string* before *count* occurrences of the delimiter *delim*. If *count* is positive, everything to the left of the final delimiter (counting from the left) is returned. If *count* is negative, everything to the right of the final delimiter (counting from the right) is returned. The function substring_index performs a case-sensitive match when searching for *delim*.
| s | MAP_KEYS(map) | Returns the keys of the *map* as an array, the order of the entries is not defined
| s | MAP_VALUES(map) | Returns the values of the *map* as an array, the order of the entries is not defined
| b m p r s h | MD5(string) | Calculates an MD5 128-bit checksum of *string* and returns it as a hex string
Copy link
Contributor

Choose a reason for hiding this comment

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

I hope | b m p r s h | is in alphabetical order too.

@NobiGo
Copy link
Contributor

NobiGo commented Feb 13, 2025

If you want this to stay this way you may need to add some kind of linter rule to enforce it. Not sure how easy it is to parse reference.md to check it, though.

+1 It would be better if it could be done this way.

@ILuffZhe
Copy link
Contributor Author

Now I get your guys point, I'll see what I can do.

@ILuffZhe ILuffZhe marked this pull request as draft February 15, 2025 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants