Skip to content

Fix ValueError on unlisted CrateDB column type identifiers - #830

Open
bgunebakan wants to merge 4 commits into
mainfrom
bilal/829-datatype-enum-reads-raise-valueerror
Open

Fix ValueError on unlisted CrateDB column type identifiers#830
bgunebakan wants to merge 4 commits into
mainfrom
bilal/829-datatype-enum-reads-raise-valueerror

Conversation

@bgunebakan

@bgunebakan bgunebakan commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary of the changes / Why this is an improvement

  • Added the five missing members to DataType.
  • Added _resolve() that returns None for identifiers unknown to this client instead of raising error.
  • Add UUID support to DefaultTypeConverter and update documentation

It is out of the scope of the issue ticket but I added a converter for DataType.UUID in this PR as well, because the enum member is introduced here.Before the round trip was asymmetric. uuid.UUID is already accepted as a parameter on the write side, but reads came back as str. DefaultTypeConverter now decodes UUID columns to uuid.UUID.

Checklist

@bgunebakan bgunebakan self-assigned this Sep 10, 2026
@bgunebakan bgunebakan added the bug label Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DataType enum: reads raise ValueError on five documented wire type IDs

1 participant