ROUND() now properly supports INTEGER and DECIMAL
Open
Nobody has claimed this yet.
2.10
reference
sql
- Dominant language
- CSS
- Stars
- 15
- Forks
- 49
- Avg merge
- 1d 13h
- Merged PRs (30d)
- 3
Description
Root document: https://www.tarantool.io/en/doc/latest/reference/reference_sql/
INTEGER and DECIMAL values passed as the first argument now will not be
cast to DOUBLE and the result will be of the same type as the first
argument. Also, the default type for the ROUND() is now DECIMAL.
Requested by @ImeevMA in https://github.com/tarantool/tarantool/commit/5273b8cbe1425cb04432cfe7c9743cb4534c9701.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Open the linked SQL reference document and locate the ROUND() entry. Update its description to cover INTEGER and DECIMAL inputs retaining the first argument's type, and note DECIMAL as the default type; confirm the published reference matches these behaviors.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- sql
- Domain
- documentation
- Issue type
- Documentation
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100