-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Query was empty error on Preview 16 #902
Comments
Can you please tell me how to reproduce this? I think this might be a regression from this PR: #841 |
It seems like a time issue and it happens intermittently. All I can do is provide a screen recording. https://www.loom.com/share/a42393267be34b6eb658460d2d05f4cc In the video, you can see it fails just after more than 20-25 secs ellipse between requests. Nothing changed with the query or anything. |
This happens in preview16.1 as well |
It just fails after being idle 🤷🏽♂️ |
The same issue. I can provide error log from the server:
Version: prisma2@2.0.0-preview016.1 |
I am unable to reproduce this. I used the GraphQL starter. Can anyone share your code so that I can try with that once? |
@pantharshit00 Here ya go - https://github.com/iRoachie/another-prisma2-bug Starter Project again. Nothing changed. |
I can reproduce this now. Seems only reproducible in with the MySQL connector, not with the Postgres or SQLite connector. Thanks again for sharing the reproduction with us :) Internal note: |
Found the bug, fixing today. The problematic method is this, but I'm just rewriting the pooling and now all connections test their status with https://github.com/prisma/quaint/blob/master/src/pool/mysql.rs#L39-L44 |
Going to be merged to prisma engines tomorrow, the quaint changes are here prisma/quaint#30 https://github.com/prisma/quaint/pull/30/files#diff-b4aea3e418ccdb71239b96952d9cddb6R351-R355 |
Fix should be in the next release on Thursday (or if compiling the core from sources). |
I'm still seeing this error on
|
The preview 17 was delayed to next Monday. I still needed to test this with the latest master, which I've been using together with MySQL for benchmarking the whole week without seeing this issue. Now I've been running the latest query engine for over ten minutes and the connections are still up with no errors. Logs after being idle for a while and then querying: Nov 23 12:47:41.959 INFO quaint::connector::metrics: query: "SELECT 1", params: [] (in 1ms)
Nov 23 12:47:41.959 INFO quaint::connector::metrics: query: "SELECT `prisma`.`Artist`.`ArtistId` FROM `prisma`.`Artist` WHERE `prisma`.`Artist`.`ArtistId` = ? ORDER BY `prisma`.`Artist`.`ArtistId` ASC LIMIT ? OFFSET ?", params: [55,1,0] (in 0ms) Where we can see the first query checking the connection status after being idle for a while, which used to be the empty query (which btw. works for psql/sqlite), but is now a You can try to find the Prisma binary from your node modules, and see the version: > ./target/release/prisma --version
b8d90fea39d266b128b4d748db5aca8505bb1026 The currently available preview 16.2 is not including the fix yet. |
I realize that Preview 16.2 doesn't include the fix, as I've been using that for the last two weeks. I'm using I'll do additional verification/testing as I can over the weekend, as well as after the official release is made on Monday before starting a new issue. |
It would still be useful if you'd give me the exact commit id from the prisma binary, so I'd know in which commit you're currently. You should get it with the |
Ok, tried it with the binary distributed through npm, so In > ./query-engine-debian-openssl-1.1.x --version
b8d90fea39d266b128b4d748db5aca8505bb1026 Been running this binary for 30 minutes against a MySQL database with no errors. Maybe you could try to uninstall prisma2 and then install the latest alpha from scratch. Might be some old npm version somewhere you're using instead of the latest. |
Seems to line up with yours. I'm running all this containerized, and have setup scripts to grab all related dependencies for global npm packages (including prisma2) as well as deps for API backend and UI. I've reset on many occasions with a full nuke script which wipes any Just tested again, and it took me about 5 clicks around and attempts to hit the API before I got the error again. |
Hi @himerus, Can you please make sure that you ran To make sure run the this:
(if you are on a different OS than Mac, change the binary name accordingly) and it should match the following hash: You might also need to update your project to utilize the new photon facade if you haven't done that already: prisma/prisma-client-js#261. To do that, run I was also unable to reproduce this now. |
Do you know if preview017 is scheduled for today, or for next Monday? I have the same issue with the findOne query and I want to know if I switch to the alpha version or wait for the preview one. |
It should be coming out today. Was postponed from last Thursday due to some last minute fixes and not having enough time. |
@pantharshit00 @pimeys Thanks for the help on this one. My report was 'correct', but it was caused because I was trying to use a global install of the Primsa CLI. So I had one (global) bumped to latest, but not the internal package in So I was on an odd edge case. I've managed to get everything updated today with the latest from Preview 17, including a couple hiccups with pinned dependencies on nexus and nexus-prisma. Once got past all that, it appears to be smooth sailing now. |
Good that it's settled for you. Happy hacking :) |
I recently updated my project to use preview016 which released today. However, intermittently there are errors being thrown.
More exactly:
I'd run a query (works), wait for couple seconds, and then re-run the same exact query, and the error would throw.
The only thing that works is to downgrade back to preview015.
The text was updated successfully, but these errors were encountered: