Repository navigation
Conversation
Generated-by: OpenAI Codex (GPT-6)
|
Hi @emecii, I havn't had to time to back to my stale PR, but maybe its something worth looking into as part of this PR? |
|
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 pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
Closes #3714
Rationale for this change
PyArrow-to-Iceberg conversion tests cover supported primitive types individually, but the expected PyArrow and Iceberg-spec Parquet physical mappings are not collected in one contract. This table-driven test covers the supported primitive mappings, including UUID, fixed binary, timestamp variants, and all three decimal physical-width bands.
Are these changes tested?
make test PYTEST_ARGS='-q'— 4,033 passed, 3 skipped, 1,571 integration-marked tests deselected by the project target.make check-licensecould not run locally because no Java runtime is installed. This PR changes one existing test file and adds no source files.Are there any user-facing changes?
No runtime behavior changes. The expanded regression contract protects type-conversion compatibility.