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

RQ: Raw Query Improvement Epic #12367

Closed
37 of 50 tasks
matthewmueller opened this issue Mar 15, 2022 · 1 comment · Fixed by prisma/quaint#361
Closed
37 of 50 tasks

RQ: Raw Query Improvement Epic #12367

matthewmueller opened this issue Mar 15, 2022 · 1 comment · Fixed by prisma/quaint#361
Assignees
Labels
kind/epic A high-level initiative that has many subtasks. team/client Issue for team Client. topic: breaking change topic: raw $queryRaw(Unsafe) and $executeRaw(Unsafe): https://www.prisma.io/docs/concepts/components/prisma-cli

Comments

@matthewmueller
Copy link
Contributor

matthewmueller commented Mar 15, 2022

Problem

Raw query support provides an important escape hatch for those instances when Prisma doesn't quite support what you need it to support. Over time, we've accumulated a number of raw query bugs. We'd like to fix these issues and start addressing them in a batch.

Goal

We do another pass over our raw query bugs. We understand and fix the ones we can. For the ones we can't, we document the limitation and look for ways to fix them in the future.

Tasks

Umbrella Issues

Issues

These issues came from the Bug Cluster Column

Confirmed Bugs

Legend:
🔴 - Blocked
🟡 - Fixable but breaking
🟢 - Fixable
🟤 - Unknown

Float input/output

🔴 Blocked by Float rework

Incorrect QE query parameter coercion

🟢 Fixable

Incorrect Client result coercion

🟡 BREAKING: Rework needed to propagate type information from QE to Client

Enum input/output

🟤 Need to be discussed, not sure if there's a solution

Geography type input

🔴 Blocked but a workaround was provided

Void output (non existing type)

🔴 Blocked by rust pg driver (according to the issue comments, to be confirmed)

Date output

🟤 We currently ignore the timezone when deserializing TimeTz type. Next step: figure out why

Unsupported native database type

🔴 Blocked by native database type support

Incorrect PG query parameter type inferring

🟡 BREAKING: Rework needed to pass in the correct query parameters types

Needs text protocol support

🔴 Blocked by Text protocol support

Wrong parameters

🟢 Fixable

Stored procedure output

⚠️ Still needs to be investigated

SQL prepared statements (wrongly escaped expressions)

🟢 Fixable - Most of those aren't bugs, users just misunderstand prepared statements.

NULLs in scalar lists

🟢 Fixable

No error code

Unconfirmed Bugs

🟤 We can hardly reproduce those issues or they're unconfirmed

New features

Not really a bug, it is something we don't support (yet)

@idolize
Copy link

idolize commented Jun 10, 2022

I've updated the issue #11454 with a repo showing a basic case of how to reproduce the issue: https://github.com/idolize/prisma-issue-11454

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/epic A high-level initiative that has many subtasks. team/client Issue for team Client. topic: breaking change topic: raw $queryRaw(Unsafe) and $executeRaw(Unsafe): https://www.prisma.io/docs/concepts/components/prisma-cli
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants