-
Notifications
You must be signed in to change notification settings - Fork 90
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
Fix support for BIGINT in SPL params #389
Fix support for BIGINT in SPL params #389
Conversation
Just noticed I need to rebase this onto the updated |
524537f
to
2ee0720
Compare
Rebase done. |
@@ -1,6 +1,6 @@ | |||
This stored proc tests output parameters - CalcLength returns string length as second output param | |||
|
|||
|Execute Ddl|!-Replace Procedure CalcLength(in name varchar(100), out strlength numeric) | |||
|Execute Ddl|!-Replace Procedure CalcLength(in name varchar(100), out strlength bigint) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As we're removing the numeric
version here - do we have some other test which is still covering the numeric
behavior?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're still covered in StoredProcWithException
.
None of the data type tests for Teradata stored price and functions are explicit enough really. I wonder if we could define some standard tests sections and use !include
s to supply the specific content for each DB sometime.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK. Seems good enough to me for now.
:+1 I'm merging this. Thank you @MMatten for the quick fix! |
…proc-params Fix support for BIGINT in SPL params
Corrected type mappings to fix #387.