Repository navigation
Conversation
|
🚀 IvorySQL-Docs Preview Ready
|
Contributor
|
One suggestion: To keep the formatting consistent with the other SQL examples on the page, please add the |
- Align the added out_param_func definition with the prompt style used by the other SQL examples on the page
Author
|
Done - added the The continuation markers follow the same style as the existing |
|
🚀 IvorySQL-Docs Preview Ready
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The last example in the CALL INTO documentation ("Output binding variables do not allow duplicate binding") invokes
out_param_func(...), but the function is never defined in the example. Readers who copy-paste the snippet hit a different error (function does not exist) instead of the intendedoutput parameter cannot be a duplicate binddemonstration.This PR adds the missing
out_param_funcdefinition (a function with an IN parameter, an OUT parameter, and a return value, so the INTO clause is valid) to both the Chinese and English documentation.This completes the remaining part of #224: the
stand_alone_funcdefinition was already added earlier, and this PR adds the other missing definition so the issue can be fully closed.Closes #224