Skip to content

Conversation

@amoeba
Copy link
Member

@amoeba amoeba commented Sep 27, 2024

So far this just does the inference step.

Rationale for this change

Support code like pa.array([uuid.uuid4()]). See #43855.

What changes are included in this PR?

  • Inference of incoming uuid.UUID objects as extension::uuid
  • Conversion of above
  • Documentation for this

Are these changes tested?

WIP.

Are there any user-facing changes?

Yes, though minor. Docs should be added here.

This doesn't implement anything at the Arrow Converter level so this change only helps us know whether an input is uuid.UUID. Converter still needs to get written to support ExtensionTypes.
@github-actions
Copy link

⚠️ GitHub issue #43855 has been automatically assigned in GitHub to PR creator.

@pitrou
Copy link
Member

pitrou commented Mar 3, 2025

@AlenkaF

@matthias-Q
Copy link

I think this will not be reviewed as long as it is marked as DRAFT, right?

@amoeba
Copy link
Member Author

amoeba commented Jul 12, 2025

Hi @matthias-Q, that's right. It's not ready for review yet, I ran out of steam. I'm leaving it open in case someone else wants to continue.

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