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.

Sign up for GitHub

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

Jump to bottom

Deprecate StoredProcedure #80

Open
jwp opened this issue Sep 6, 2016 · 1 comment
Open

Deprecate StoredProcedure #80

jwp opened this issue Sep 6, 2016 · 1 comment
Labels
v1.4

Comments

@jwp
Copy link
Contributor

jwp commented Sep 6, 2016

There should probably be some test entries exercising [parts of] StoredProcedure's type resolution. A recent issue revealed the lack of testing around OID entries. db.proc() generates queries based on procedure parameters and it needs an OID -> Name mapping if it's a recognized type. Currently, there are no automated checks for this.
@jwp
Copy link
Contributor Author

jwp commented Feb 10, 2023

Alternatively, add deprecation warnings for proc and encourage prepare usage.

@jwp jwp added v1.4 and removed cake typio labels Feb 10, 2023
@jwp jwp changed the title StoredProcedure tests Deprecate StoredProcedure Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
v1.4
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jwp