Skip to content

fix(client): preserve prepared null parameter types - #18

Merged
fvaleye merged 1 commit into
mainfrom
fl/preserve-prepared-null-types
Jul 31, 2026
Merged

fix(client): preserve prepared null parameter types#18
fvaleye merged 1 commit into
mainfrom
fl/preserve-prepared-null-types

Conversation

@fvaleye

@fvaleye fvaleye commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Preserve the prepared parameter type when converting Python None values to Arrow arrays.
  • Cover both positional and named parameter bindings with regression tests.

PyArrow infers [None] as Arrow's null type. Flight SQL prepared statements provide an expected bind parameter schema, so sending that inferred type can make the parameter batch differ from the schema the server prepared.

The client now builds null arrays with the prepared field type when that type is known. It keeps inference for non-null values and for named parameters absent from the prepared schema, preserving the existing permissive behavior.

@fvaleye
fvaleye force-pushed the fl/preserve-prepared-null-types branch from 2d8f9b5 to a4d05ce Compare July 31, 2026 08:07
@fvaleye
fvaleye requested a review from redox July 31, 2026 08:25
@fvaleye
fvaleye force-pushed the fl/preserve-prepared-null-types branch from a4d05ce to d2e1de6 Compare July 31, 2026 08:32
@fvaleye
fvaleye requested review from leo-altertable and utay July 31, 2026 08:48
@fvaleye
fvaleye merged commit bb0ebfd into main Jul 31, 2026
6 checks passed
@fvaleye
fvaleye deleted the fl/preserve-prepared-null-types branch July 31, 2026 10:21
fvaleye pushed a commit that referenced this pull request Jul 31, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.3.3](v0.3.2...v0.3.3)
(2026-07-31)


### Bug Fixes

* **client:** preserve prepared null parameter types
([#18](#18))
([bb0ebfd](bb0ebfd))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants