Skip to content

fix: binary string concat#20787

Open
theirix wants to merge 4 commits intoapache:mainfrom
theirix:binary-to-binaryconcat
Open

fix: binary string concat#20787
theirix wants to merge 4 commits intoapache:mainfrom
theirix:binary-to-binaryconcat

Conversation

@theirix
Copy link
Contributor

@theirix theirix commented Mar 7, 2026

Which issue does this PR close?

Rationale for this change

There is no support for concatenating binary strings.

There are two ways:

  1. Cast binaries to text and then concatenate text type (most common behaviour across DBs as discussed in the ticket)
  2. Concatenate binaries and provide binary type (only for Spark)

This PR goes with a way (1). We could customise the operator/UDF per-dialect, but it is not currently supported.

What changes are included in this PR?

  • Support binary concatenation via a pipe operator
  • Support binary concatenation in concat UDF

Are these changes tested?

Added SLT tests

Are there any user-facing changes?

No

@github-actions github-actions bot added logical-expr Logical plan and expressions physical-expr Changes to the physical-expr crates sqllogictest SQL Logic Tests (.slt) functions Changes to functions implementation labels Mar 7, 2026
@theirix theirix marked this pull request as ready for review March 7, 2026 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

functions Changes to functions implementation logical-expr Logical plan and expressions physical-expr Changes to the physical-expr crates sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Binary string (BYTEA, Binary) concatenation

1 participant