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

Support sql prepare for show statements #3972

Closed
knz opened this issue Jan 26, 2016 · 4 comments
Closed

Support sql prepare for show statements #3972

knz opened this issue Jan 26, 2016 · 4 comments
Assignees

Comments

@knz
Copy link
Contributor

knz commented Jan 26, 2016

This is a followup to #3958 and #3819.
The jdbc driver generally talks to the database by means of prepare/bind/execute, including for SHOW statements. But at this moment this fails with cockroach with:

ERROR: sql/plan.go:161: prepare statement not supported: *parser.Show
@knz knz added the SQL label Jan 26, 2016
@JackKrupansky
Copy link

Prepared statements are not currently even listed on the Beta/1.0 roadmap.

@maddyblue maddyblue self-assigned this Jan 26, 2016
@maddyblue
Copy link
Contributor

We've already begun implementing prepared statements (#3482) and are attempting to complete them (#3958).

@maddyblue
Copy link
Contributor

Postgres doesn't appear to support prepared SHOW statements, so I'm inclined to not support this. Can you give me more details about the exact statement, and can you verify it works against a real postgres server?

@knz
Copy link
Contributor Author

knz commented Jan 29, 2016

Ok I tested again and the SHOW statements goes through properly. Thanks.

@knz knz closed this as completed Jan 29, 2016
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

3 participants