Skip to content

fix: replace fetch_arrow_table with to_arrow_table#6146

Merged
BubbleCal merged 1 commit intomainfrom
yang/update-duckdb-integration-test
Mar 10, 2026
Merged

fix: replace fetch_arrow_table with to_arrow_table#6146
BubbleCal merged 1 commit intomainfrom
yang/update-duckdb-integration-test

Conversation

@BubbleCal
Copy link
Copy Markdown
Contributor

fix CI error: FAILED python/tests/test_integration.py::test_duckdb_pushdown_extension_types - _duckdb.Error: DeprecationWarning: fetch_arrow_table() is deprecated, use to_arrow_table() instead.

@github-actions github-actions Bot added bug Something isn't working python labels Mar 10, 2026
@github-actions
Copy link
Copy Markdown
Contributor

LGTM. Straightforward replacement of deprecated fetch_arrow_table() with to_arrow_table() across all call sites in the test file. No issues.

@XuQianJin-Stars
Copy link
Copy Markdown
Contributor

+1

@Xuanwo
Copy link
Copy Markdown
Collaborator

Xuanwo commented Mar 10, 2026

Thank you for working on this, I'm just working on this now and found that you have fixed it.

@BubbleCal BubbleCal merged commit 31de8d4 into main Mar 10, 2026
14 checks passed
@BubbleCal BubbleCal deleted the yang/update-duckdb-integration-test branch March 10, 2026 07:49
westonpace pushed a commit that referenced this pull request Mar 18, 2026
fix CI error: `FAILED
python/tests/test_integration.py::test_duckdb_pushdown_extension_types -
_duckdb.Error: DeprecationWarning: fetch_arrow_table() is deprecated,
use to_arrow_table() instead.`
westonpace added a commit that referenced this pull request Mar 18, 2026
## Summary

Cherry-picks bug fixes onto `release/v3.0` for the v3.0.1 patch release:

- **#6160** - fix: handle `DataType::Null` in `adjust_child_validity` to
prevent panic
- **#6187** - fix: handle nullable validity layers without def levels
- **#6143** - fix: prevent duplicate manifest entries from concurrent
table creation
- **#6212** - chore: bump lz4_flex patch versions
- **#6146** - fix: replace fetch_arrow_table with to_arrow_table

## Test plan

- CI passes on cherry-picked commits (both PRs were already merged and
tested on main)

---------

Co-authored-by: Will Jones <willjones127@gmail.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Xuanwo <github@xuanwo.io>
Co-authored-by: Jonathan Hsieh <jon@lancedb.com>
Co-authored-by: BubbleCal <bubble-cal@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants