Skip to content

Conversation

carlopi
Copy link
Contributor

@carlopi carlopi commented Sep 11, 2024

Moving the error messages in cases of missing enum case to be more verbose, given this has basically no additional cost but grants some more information on what the system is doing at the moment the exception is thrown.

Before (using a bug that will be independently fixed in the meantime as example):

D LOAD '~/.duckdb/extensions/v1.0.0/osx_arm64/spatial.duckdb_extension';
Not implemented Error: Enum value: '' not implemented

After:

D LOAD '~/.duckdb/extensions/v1.0.0/osx_arm64/spatial.duckdb_extension';
Not implemented Error: Enum value: '' not implemented in FromString<ExtensionABIType>

@carlopi carlopi added the Draft label Sep 12, 2024
@carlopi carlopi force-pushed the improve_error_on_enums branch from 943cc5d to ef3c4bd Compare September 12, 2024 08:58
@duckdb-draftbot duckdb-draftbot marked this pull request as draft September 12, 2024 09:01
@carlopi carlopi marked this pull request as ready for review September 12, 2024 09:08
@Mytherin Mytherin merged commit 88a4c1e into duckdb:main Sep 12, 2024
43 checks passed
@Mytherin
Copy link
Collaborator

Thanks!

github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Sep 25, 2024
Improve error on enums (duckdb/duckdb#13885)
[Python] Add `python_scan_all_frames` to opt-in to scanning all frames (< 1.1 behavior) (duckdb/duckdb#13896)
github-actions bot added a commit to duckdb/duckdb-r that referenced this pull request Sep 25, 2024
Improve error on enums (duckdb/duckdb#13885)
[Python] Add `python_scan_all_frames` to opt-in to scanning all frames (< 1.1 behavior) (duckdb/duckdb#13896)

Co-authored-by: krlmlr <krlmlr@users.noreply.github.com>
@carlopi carlopi deleted the improve_error_on_enums branch February 4, 2025 08:18
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.

2 participants