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 MySqlCommandBuilder.DeriveParameters for MySQL Server 5.0 #1126

Open
dxmi opened this issue Feb 2, 2022 · 2 comments
Open

Support MySqlCommandBuilder.DeriveParameters for MySQL Server 5.0 #1126

dxmi opened this issue Feb 2, 2022 · 2 comments

Comments

@dxmi
Copy link

dxmi commented Feb 2, 2022

Is your feature request related to a problem? Please describe.
We need support for legacy MySQL Server 5.0 similar to the level the MySql.Data provides.

Specifically, we use the MySql.Data.MySqlClient.MySqlCommandBuilder.DeriveParameters method to obtain the schema of a stored procedure.
The MySqlConnector.MySqlCommandBuilder.DeriveParameters throws an exception for server versions less than 5.5.3 due to the absence of "information_schema.routines".

Describe the solution you'd like
As I see, some fallback code using "mysql.proc" already exists in the CachedProcedure.FillAsync method. Use this code to derive parameters instead of throwing an exception.

Describe alternatives you've considered
We'd like to avoid re-implementing procedure parameter parsing on our side.

@bgrainger
Copy link
Member

If you'd like to open a PR to add the support you need (with code that is entirely your original contribution and able to be licensed to this project), I'd be happy to review it.

@bgrainger
Copy link
Member

(It looks like the earliest-available MySQL Docker image is mysql:5.5.40, which would make developing and testing this difficult.)

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

No branches or pull requests

2 participants