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
sql: add missing PostgreSQL built-in functions #26586
Conversation
Review status: complete! 0 of 0 LGTMs obtained (and 1 stale) pkg/sql/logictest/testdata/logic_test/builtin_function, line 42 at r1 (raw file):
This is also sensitive to SQL keywords, right? Do we need a test of that? Comments from Reviewable |
Review status: complete! 0 of 0 LGTMs obtained (and 1 stale) pkg/sql/logictest/testdata/logic_test/builtin_function, line 42 at r1 (raw file): Previously, justinj (Justin Jaffray) wrote…
Nice catch. Done. Comments from Reviewable |
TFYR! bors r+ |
Build failed |
Release note (sql change): CockroachDB now supports the built-in functions `bit_length()`, `quote_ident()`, `quote_literal()`, `quote_nullable()`, and the aliases `char_length()` and `character_length()` for `length()`, for compatibility with PostgreSQL.
github API flake (502 server error) bors r+ |
26586: sql: add missing PostgreSQL built-in functions r=knz a=knz Fixes #26406. Release note (sql change): CockroachDB now supports the built-in functions `bit_length()`, `quote_ident()`, `quote_literal()`, `quote_nullable()`, and the aliases `char_length()` and `character_length()` for `length()`, for compatibility with PostgreSQL. Co-authored-by: Raphael 'kena' Poss <knz@cockroachlabs.com>
Build succeeded |
Fixes #26406.
Release note (sql change): CockroachDB now supports the built-in
functions
bit_length()
,quote_ident()
,quote_literal()
,quote_nullable()
, and the aliaseschar_length()
andcharacter_length()
forlength()
, for compatibility withPostgreSQL.