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 prepared statement parameter type inference #4700

Closed
avantgardnerio opened this issue Dec 21, 2022 · 1 comment
Closed

Support prepared statement parameter type inference #4700

avantgardnerio opened this issue Dec 21, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@avantgardnerio
Copy link
Contributor

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

When implementing FlightSQL, it was observed that the prepare ... syntax that our prepared statement code expects is not present. This is problematic, because there is no syntax from which to find type declarations. When comparing to how the JDBC client behaves, it appears to expect parameter types to be inferred.

Describe the solution you'd like

  1. Don't require prepare ... syntax (just allow a regular query to be prepared as a statement)
  2. Infer the parameter types
  3. Add a method to return inferred types

Describe alternatives you've considered

  1. Have all existing JDBC code rewritten to support the model as present in DataFusion today
  2. Do not support prepared statements
  3. Always return a generic type like "String" or "Object" and coerce
@avantgardnerio avantgardnerio added the enhancement New feature or request label Dec 21, 2022
@avantgardnerio
Copy link
Contributor Author

Accidental duplicate of #4683

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant