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

Document the new string functions #697

Closed
hengm3467 opened this issue Mar 30, 2023 · 3 comments · Fixed by #847
Closed

Document the new string functions #697

hengm3467 opened this issue Mar 30, 2023 · 3 comments · Fixed by #847
Assignees
Labels
documentation Improvements or additions to documentation
Milestone

Comments

@hengm3467
Copy link
Collaborator

Related code PR

risingwavelabs/risingwave#8767

Which part(s) of the docs might be affected or should be updated? And how?

SQL - Functions - String

Please experiment with creating docs for these functions using chatGPT

Adds the following functions:

chr(integer) -> varchar
starts_with(varchar, varchar) -> boolean
initcap(varchar) -> varchar
lpad(varchar, integer) -> varchar
lpad(varchar, integer, varchar) -> varchar
rpad(varchar, integer) -> varchar
rpad(varchar, integer, varchar) -> varchar
reverse(varchar) -> varchar
strpos(varchar, varchar) -> integer
to_ascii(varchar) -> varchar
to_hex(integer) -> varchar
to_hex(bigint) -> varchar

Reference

Please reference PG's docs for these string functions.

@hengm3467 hengm3467 added the documentation Improvements or additions to documentation label Mar 30, 2023
@hengm3467 hengm3467 added this to the v0.1.18 milestone Mar 30, 2023
@xiangjinwu
Copy link
Contributor

xiangjinwu commented Apr 6, 2023

Let's hold on this until next version, as we have found several bugs in them

@hengm3467 hengm3467 removed this from the v0.18.0 milestone Apr 10, 2023
@CharlieSYH CharlieSYH added this to the v0.19.0 milestone Apr 18, 2023
@CharlieSYH
Copy link
Contributor

@bernscode Please continue to update this doc PR as these code PRs are merged for 0.19.0:
risingwavelabs/risingwave#8970
risingwavelabs/risingwave#9000

@xiangjinwu
Copy link
Contributor

FYI known issues have been fixed. They should be good to go in v0.19.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants