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.

Sign up for GitHub

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

Jump to bottom

IncorrectUsage error when batching update statements. #25

Open
naushadh opened this issue Mar 29, 2019 · 2 comments
Open

IncorrectUsage error when batching update statements. #25

naushadh opened this issue Mar 29, 2019 · 2 comments
Assignees
@NCrashed
Labels
bug

Comments

@naushadh
Copy link

Issue

  • Under high load (upto 512 concurrent requests) in a web app, postgres-wire keeps running into IncorrectUsage: "Expected DataRow message, but got ReadyForQuery" error for a batch of update statements.

  • Issue does NOT manifest when the number of concurrent requests is low.

  • Related project and workaround I had to apply: TechEmpower/FrameworkBenchmarks@a707305#diff-18e221ebfc6aeb9c7554015904f0c042

Is this expected? Am I doing something wrong?
@NCrashed NCrashed self-assigned this Apr 3, 2019
@NCrashed NCrashed added the bug label Apr 3, 2019
@NCrashed
Copy link
Collaborator

NCrashed commented Apr 3, 2019

Thank you for the report! I will investigate this. Also, it will be really helpful if you have a code example that reproduces this issue.

naushadh added a commit to naushadh/postgres-wire-issue25 that referenced this issue Apr 7, 2019
@naushadh
Initial commit.
cd90f1f 
- Created miniature application based on issue experienced here: postgres-haskell/postgres-wire#25
- Added dockerfile to mimic TFB setup of using Postgres 9.5 with randomly seeded range data for the World table.
- Added Makefile to help abstract some common tasks.
@naushadh
Copy link
Author

naushadh commented Apr 7, 2019

@NCrashed Created a sample project that focuses and reproduces this issue:
https://github.com/naushadh/postgres-wire-issue25

See out.log for test reproduction output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@NCrashed NCrashed

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@naushadh @NCrashed