Skip to content

Conversation

pdet
Copy link
Contributor

@pdet pdet commented Oct 28, 2024

PyArrow 18 comes with the canonical extensions pre-registered. Hence, this new behavior breaks a lot of our test suit that would take care of registering/ensuring correctness. This PR fixes our CI by adjusting our tests.

@duckdb-draftbot duckdb-draftbot marked this pull request as draft October 29, 2024 08:35
@pdet pdet marked this pull request as ready for review October 29, 2024 08:35
@carlopi
Copy link
Contributor

carlopi commented Oct 29, 2024

This looks to make sense to me, one thing to consider, @pdet, can you trigger the whole set of Python CI on your fork?
Basically to check this works across the wider range of configurations?

It should be a matter of going to pdet/duckdb, "Action" tab, then select Python, then trigger it selecting "arrow_18" as the branch. Thanks (this is basically to cut some latency, since otherwise we will discover that only tonight)

@duckdb-draftbot duckdb-draftbot marked this pull request as draft October 29, 2024 12:20
@pdet pdet marked this pull request as ready for review October 29, 2024 12:20
@Mytherin Mytherin merged commit 1813201 into duckdb:main Oct 29, 2024
19 checks passed
github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Nov 3, 2024
Unexpected result comparing blob (duckdb/duckdb#14604)
CI: Add repo name to labelling script (duckdb/duckdb#14612)
CI: Fix labelling bot (duckdb/duckdb#14611)
Test fixes for new arrow release (duckdb/duckdb#14593)
github-actions bot added a commit to duckdb/duckdb-r that referenced this pull request Nov 3, 2024
Unexpected result comparing blob (duckdb/duckdb#14604)
CI: Add repo name to labelling script (duckdb/duckdb#14612)
CI: Fix labelling bot (duckdb/duckdb#14611)
Test fixes for new arrow release (duckdb/duckdb#14593)

Co-authored-by: krlmlr <krlmlr@users.noreply.github.com>
@pdet pdet deleted the arrow_18 branch November 27, 2024 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants