Skip to content

Conversation

@leoyvens
Copy link
Contributor

Which issue does this PR close?

What changes are included in this PR?

Adds parse_hex_as_fixed_size_binary parser option.

Are these changes tested?

There is an SLT test.

Are there any user-facing changes?

The configuration defaults to false, no breaking change. I'm not sure if docs updates are required.

@github-actions github-actions bot added sql SQL Planner core Core DataFusion crate sqllogictest SQL Logic Tests (.slt) common Related to common crate labels Apr 11, 2025
@leoyvens leoyvens force-pushed the parse_hex_as_fixed_size_binary branch 2 times, most recently from b6366ad to 72dc068 Compare April 11, 2025 18:48
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Apr 11, 2025
@leoyvens leoyvens force-pushed the parse_hex_as_fixed_size_binary branch from 72dc068 to adda058 Compare April 11, 2025 18:49
@leoyvens leoyvens force-pushed the parse_hex_as_fixed_size_binary branch from adda058 to 540ca95 Compare April 11, 2025 18:51
@alamb
Copy link
Contributor

alamb commented Apr 12, 2025

I have an alternate suggestion here: #15686 (comment)

@alamb
Copy link
Contributor

alamb commented May 1, 2025

I believe we went with an alternate approach in

@alamb alamb closed this May 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

common Related to common crate core Core DataFusion crate documentation Improvements or additions to documentation sql SQL Planner sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow parsing byte literals as FixedSizeBinary

2 participants