Skip to content

Fix type of ExecutionOptions::time_zone - #16569

Merged
findepi merged 1 commit into
apache:mainfrom
findepi:findepi/fix-type-of-executionoptions-time-zone-aa41b0
Jun 27, 2025
Merged

Fix type of ExecutionOptions::time_zone#16569
findepi merged 1 commit into
apache:mainfrom
findepi:findepi/fix-type-of-executionoptions-time-zone-aa41b0

Conversation

@findepi

@findepifindepi commented Jun 26, 2025

Copy link
Copy Markdown
Member

Previously the field was optional. However, when set to a None value, it would trigger invalid behavior, with timestamp with time zone type being parsed as if it was just timestamp type.

Previously the field was optional. However, when set to a None value, it
would trigger invalid behavior, with `timestamp with time zone` type
being parsed as if it was just `timestamp` type.
@github-actionsgithub-actionsBot added sql SQL Planner common Related to common crate labels Jun 26, 2025
@alambalamb added the api change Changes the API exposed to users of the crate label Jun 26, 2025
@alamb

Copy link
Copy Markdown
Contributor

Thank you @findepi

@findepi
findepi merged commit 9c6d6ee into apache:mainJun 27, 2025
@findepi
findepi deleted the findepi/fix-type-of-executionoptions-time-zone-aa41b0 branch June 27, 2025 13:50
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api changeChanges the API exposed to users of the cratecommonRelated to common cratesqlSQL Planner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@findepi@alamb