-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat: Add UUID and TIME_UUID as feature types (#5885) #5951
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Changes from all commits
9a8c4d4
0cc99dc
ddd33b0
6876dea
53c998b
1323ae2
26198a6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🔴 The Root Cause and ImpactBefore this PR, UUID values were stored as This also breaks the ODFV pandas/substrait transformation path in if initial_response_arrow is None:
initial_response_arrow = initial_response.to_arrow() # crashes hereAny Actual: Expected: UUID values should be converted to strings before creating the Arrow table, consistent with the mapping (Refers to lines 99-107) Was this helpful? React with 👍 or 👎 to provide feedback. |
Uh oh!
There was an error while loading. Please reload this page.