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

sketch stored procedures #750

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

robmwalsh
Copy link
Contributor

A sketch of what stored procedures (and maybe table valued functions for SQL server?) might look like, essentially mandraulically adapted from FunctionDef with an adapted type tag allowing for returning tables.

This is intended as a basis for @skmuiruri to work on. @jczuchnowski would you mind casting your eyes over this to see if it's headed down the right track before he gets too involved?

Todo:

  • Make sure you can't stick a stored procedure where it doesn't belong! I haven't used them in anger so not really sure if it's legit to use them in other statements
  • Rendering - base on Function rendering
  • Add a way to actually execute the stored procedure - Functions need to be wrapped in a select, stored procedures should be able to be executed without a select (and possibly not within one?)
  • Tests - probably also based on function rendering
  • Some examples/docs on usage wouldn't go astray!

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

Successfully merging this pull request may close these issues.

None yet

1 participant