You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "hedger9487 (via GitHub)" <gi...@apache.org> on 2026/08/27 09:12:17 UTC
[PR] feat: Support passing kwargs to polars.scan_iceberg in Table.to_polars [iceberg-python]
hedger9487 opened a new pull request, #3868:
URL: https://github.com/apache/iceberg-python/pull/3868
Closes #3128
# Rationale for this change
Currently, `Table.to_polars()` does not accept keyword arguments and calls `polars.scan_iceberg(self)` directly without forwarding options. This prevents users from passing crucial scan configurations such as `storage_options` (cloud credentials for S3/GCS/Azure), `snapshot_id` (time-travel queries), or `reader_override`.
This change updates `Table.to_polars()` to accept `**kwargs: Any` and forward them to `polars.scan_iceberg(self, **kwargs)`.
## Are these changes tested?
Yes, tested with:
- `tests/table/test_init.py::test_table_to_polars_forwards_kwargs`: Unit test verifying exact forwarding of keyword arguments (both empty and populated).
- `tests/table/test_init.py::test_table_to_polars_e2e`: End-to-end integration test verifying real Polars scan execution with an Iceberg table, both without parameters and with `snapshot_id`.
- All 115 tests in `tests/table/test_init.py` and 468 tests in `tests/table/` pass cleanly.
- All pre-commit static analysis checks (`uv run prek run -a`) pass 100%.
## Are there any user-facing changes?
Yes, `Table.to_polars()` now accepts `**kwargs` forwarded to `polars.scan_iceberg`, enabling users to supply `storage_options`, `snapshot_id`, `reader_override`, etc.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] feat: Support passing kwargs to polars.scan_iceberg in Table.to_polars [iceberg-python]
Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu merged PR #3868:
URL: https://github.com/apache/iceberg-python/pull/3868
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] feat: Support passing kwargs to polars.scan_iceberg in Table.to_polars [iceberg-python]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #3868:
URL: https://github.com/apache/iceberg-python/pull/3868#issuecomment-6049672589
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that's incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] feat: Support passing kwargs to polars.scan_iceberg in Table.to_polars [iceberg-python]
Posted by "hedger9487 (via GitHub)" <gi...@apache.org>.
hedger9487 commented on PR #3868:
URL: https://github.com/apache/iceberg-python/pull/3868#issuecomment-6065476054
Rebased onto latest `main` so the newly required CI checks (`markdown-link-check`, `python-ci-required`, `python-integration-required`) can run. cc @Fokko
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org