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

Cleanup convenience functions in IDataType #56375

Merged

Conversation

rschu1ze
Copy link
Member

@rschu1ze rschu1ze commented Nov 6, 2023

A small cleanup of the convenience functions in IDataType.

Also moves the docs of string similarity functions from the string search function docs to the general string function docs + documents stringJaccardIndexUTF8() (see comments in #55126)

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-not-for-changelog This PR should not be mentioned in the changelog label Nov 6, 2023
@robot-ch-test-poll3
Copy link
Contributor

robot-ch-test-poll3 commented Nov 6, 2023

This is an automated comment for commit 32a3b1b with description of existing statuses. It's updated for the latest CI running

⏳ Click here to open a full report in a separate page

Successful checks
Check nameDescriptionStatus
Mergeable CheckChecks if all other necessary checks are successful✅ success
Check nameDescriptionStatus
CI runningA meta-check that indicates the running CI. Normally, it's in success or pending state. The failed status indicates some problems with the PR⏳ pending

@alexey-milovidov alexey-milovidov self-assigned this Nov 6, 2023
@rschu1ze rschu1ze merged commit 4db2e25 into ClickHouse:master Nov 7, 2023
5 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-not-for-changelog This PR should not be mentioned in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants