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

MySQL should support stored procedures #31

Closed
pezipink opened this issue Jan 29, 2014 · 3 comments
Closed

MySQL should support stored procedures #31

pezipink opened this issue Jan 29, 2014 · 3 comments

Comments

@pezipink
Copy link
Contributor

The current MySQL provider does not implement the ISqlProvider.GetSprocs function.

The implementation should use the MySQL metadata capabilities to establish the names and parameters of stored procedures. (Probably using INFORMATION_SCHEMA, or maybe the Connection.GetSchema can tell you about sprocs)

The more tricky bit is working out what columns the sproc returns. We can only really support simple procedures that don't return multiple or different recordsets depending on branching logic. OUT parameters are currently not supported so those ones should be ignored.

The SQL Server implementation (which is somewhat of a mess currently) executes the proc with some default parameters along with ``CommandBehaviour.SchemaOnly` to establish this information. I don't know the capabilities of MySQL, perhaps there is a nicer way to achieve this.

Once you are able to return a list of sproc data, the rest should just all magically work!

@colinbull
Copy link
Collaborator

@pezipink See #83 no support yet but should be easier now.

colinbull referenced this issue in colinbull/SqlProvider Aug 1, 2014
@pezipink
Copy link
Contributor Author

@colinbull we can close this now right?

@colinbull
Copy link
Collaborator

I think so, although if you got better schema to test it against other than my crappy one that would probably be good.

-----Original Message-----
From: "Ross McKinlay" notifications@github.com
Sent: ‎15/‎09/‎2014 14:27
To: "fsprojects/SQLProvider" SQLProvider@noreply.github.com
Cc: "Colin" colinbul@googlemail.com
Subject: Re: [SQLProvider] MySQL should support stored procedures (#31)

@colinbull we can close this now right?

Reply to this email directly or view it on GitHub.=

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

No branches or pull requests

2 participants