Skip to content

FIX: Support Arrow View types in bulk copy - #717

Draft
gargsaumya wants to merge 1 commit into
mainfrom
saumya/gh-708
Draft

FIX: Support Arrow View types in bulk copy#717
gargsaumya wants to merge 1 commit into
mainfrom
saumya/gh-708

Conversation

@gargsaumya

@gargsaumya gargsaumya commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Work Item / Issue Reference

GitHub Issue: #708
ADO: AB#47279


Summary

Adds an end-to-end regression test for passing a Polars DataFrame with an Arrow string_view column directly to Cursor.bulkcopy_arrow. The test asserts the exported Arrow type and verifies values and NULLs round-trip through SQL Server without calling DataFrame.to_arrow().

Adds Polars to the test requirements so the regression runs in CI instead of being skipped.

Depends on microsoft/mssql-rs#259 and publication of the fixed mssql_py_core 0.1.9 wheel. After publication, this branch must update eng/versions/mssql-py-core.version from 0.1.8 to 0.1.9 before it is ready to merge.

Validation

  • Live SQL Server: TestBulkcopyArrowLive::test_polars_string_view_c_stream passed
  • Black check passed for tests/test_024_bulkcopy_arrow.py
  • git diff --check passed

@github-actions github-actions Bot added the pr-size: small Minimal code update label Aug 13, 2026
@gargsaumya gargsaumya changed the title FIX: Add Polars Arrow bulk copy regression coverage FIX: Support Arrow View types in bulk copy- #259 Aug 13, 2026
@gargsaumya gargsaumya changed the title FIX: Support Arrow View types in bulk copy- #259 FIX: Support Arrow View types in bulk copy Aug 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-size: small Minimal code update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant