-
Couldn't load subscription status.
- Fork 2.9k
Update floor-transact-sql.md #10202
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
Update floor-transact-sql.md #10202
Conversation
Changed return type for `decimal(p, s)` to `decimal(p, 0)`, rather than `decimal(38, s)`.
|
@sqlHippo : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
@sqlHippo : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 096040e: ✅ Validation status: passed
For more details, please refer to the build report. |
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.
Pull Request Overview
This PR corrects the documentation for SQL Server's FLOOR function return type. The change fixes an inaccuracy in the documented return type for decimal(p, s) inputs to properly reflect that the function returns decimal(p, 0) instead of the incorrectly documented decimal(38, s).
- Updated return type documentation for decimal inputs to the FLOOR function
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
Can you review the proposed changes? Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
|
This is already in discussion internally. #reassign:rwestMSFT |
|
Duplicate of #10177. |
Changed return type for
decimal(p, s)todecimal(p, 0), rather thandecimal(38, s).Demonstation: https://dbfiddle.uk/Ai79GDn8