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

Copy command failing via odyssey #554

Open
evkuzin opened this issue Jan 12, 2024 · 6 comments
Open

Copy command failing via odyssey #554

evkuzin opened this issue Jan 12, 2024 · 6 comments

Comments

@evkuzin
Copy link

evkuzin commented Jan 12, 2024

Hi guys. Odyssey doesn't seem to support copy command. Is it known? Are there any plans? Is it a bug?

against postgres directly:

$ for i in {1..1000}; do echo "run_id_${i},task_id_${i},Some random ${i}th text"; done | psql postgresql://postgres@db:5433/test -c "COPY testing FROM STDIN (FORMAT csv);";
COPY 1000

Via odysssey

$ for i in {1..1000}; do echo "run_id_${i},task_id_${i},Some random ${i}th text"; done | psql postgresql://postgres@db:5432/test -c "COPY testing FROM STDIN (FORMAT csv);";
ERROR:  unexpected message type 0x00 during COPY from stdin
CONTEXT:  COPY testing, line 167
ERROR:  unexpected message type 0x00 during COPY from stdin
CONTEXT:  COPY testing, line 167
FATAL:  terminating connection because protocol synchronization was lost

@x4m in case you know. Thx

@x4m
Copy link
Collaborator

x4m commented Jan 14, 2024

AFAIK this was fixed in 6ea14ee

@evkuzin
Copy link
Author

evkuzin commented Jan 15, 2024

Would have been nice, but I'm using binary built with this and it is much older than your commit. The error is still there 🤷

@evkuzin
Copy link
Author

evkuzin commented Jan 15, 2024

@reshke seems like your fix wasn't enough to fix COPY :(

@x4m
Copy link
Collaborator

x4m commented Jan 16, 2024

Do you observer reproduction on actual master?

I'm using binary built with this and it is much older than your commit.

Do you mean newer?

@evkuzin
Copy link
Author

evkuzin commented Jan 16, 2024

🤦 Yup, I meant newer 🤣

@evkuzin
Copy link
Author

evkuzin commented Jan 24, 2024

Update.
The issue is with

pool_reserve_prepared_statement yes

in the config
Without it, everything works like a charm.

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

No branches or pull requests

2 participants