Skip to content
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

fix: ignore extra fields in Postgres describe parsing #2670

Merged
merged 5 commits into from Jul 31, 2023
Merged

Conversation

abonander
Copy link
Collaborator

closes #2626 #2622 #2587

Credit to @mrl5 for the fix, I just wanted to get this merged today without any extra back-and-forth.

mrl5 and others added 4 commits July 17, 2023 21:51
```
error: error occurred while decoding column 0: data did not match any variant of untagged enum Explain at line 3 column 1
Error:    --> tests/postgres/macros.rs:103:15
    |
103 |     let row = sqlx::query!(r#"CALL forty_two(null)"#)
    |               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this error originates in the macro `$crate::sqlx_macros::expand_query` which comes from the expansion of the macro `sqlx::query` (in Nightly builds, run with -Z macro-backtrace for more info)

error: could not compile `sqlx` (test "postgres-macros") due to previous error
```
@abonander abonander merged commit 487b89a into main Jul 31, 2023
58 checks passed
@abonander abonander deleted the issue-2622 branch July 31, 2023 21:52
@Thomasdezeeuw Thomasdezeeuw mentioned this pull request Sep 18, 2023
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.

None yet

2 participants