Skip to content

feat: improved type mismatch error messages#17

Merged
Mytherin merged 5 commits intoduckdb:mainfrom
Mause:feature/improved-error-messages
Nov 11, 2022
Merged

feat: improved type mismatch error messages#17
Mytherin merged 5 commits intoduckdb:mainfrom
Mause:feature/improved-error-messages

Conversation

@Mause
Copy link
Contributor

@Mause Mause commented Nov 6, 2022

Fixes #15

Not sure if it's possible to assert on error messages in our SQL tests?

Copy link
Contributor

@Mytherin Mytherin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Looks great. One comment:

@Mytherin
Copy link
Contributor

Mytherin commented Nov 6, 2022

Fixes #15

Not sure if it's possible to assert on error messages in our SQL tests?

We recently added syntax for this, it is as follows:

statement error
<query>
----
<expected_error>

@Mause Mause requested review from Mytherin and hannes November 8, 2022 14:42
@Mytherin Mytherin merged commit 14ea827 into duckdb:main Nov 11, 2022
@Mause Mause deleted the feature/improved-error-messages branch November 13, 2022 12:08
ak2k pushed a commit to ak2k/duckdb-sqlite that referenced this pull request Jun 7, 2025
feat: improved type mismatch error messages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SQLite: Expected integer, got something else

2 participants