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

[Doc]modify pmod radians random #1940

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[Doc]modify pmod radians random #1940

wants to merge 2 commits into from

Conversation

xiaozhuoyang
Copy link
Contributor

@xiaozhuoyang xiaozhuoyang commented Jan 24, 2025 •
edited
Loading

Versions

  • dev
  • 3.0
  • 2.1
  • 2.0

Languages

  • Chinese
  • English

Docs Checklist

  • Checked by AI
  • Test Cases Built

@KassieZ KassieZ requested a review from morrySnow January 24, 2025 09:07
wangtianyi2004
wangtianyi2004 reviewed Jan 24, 2025
View reviewed changes
...gin-content-docs/current/sql-manual/sql-functions/scalar-functions/numeric-functions/pmod.md Outdated
## 描述

返回模运算 x mod y 在模系中的最小正数解。具体而言，通过计算 (x % y + y) % y 得出结果。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

将具体而言改成即
即通过计算 (x % y + y) % y 得出结果。

...n-content-docs/current/sql-manual/sql-functions/scalar-functions/numeric-functions/random.md Outdated

- 不传参时：返回0-1之间的随机数。

- 传入单个参数`seed`时：根据传入的种子值`seed`，产生相同的随机数。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

说明返回 0-1 之间的随机数

...n-content-docs/current/sql-manual/sql-functions/scalar-functions/numeric-functions/random.md Outdated
`DOUBLE random()`
返回0-1之间的随机数。
```sql
RANDOM()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RANDOM() -- 返回 0-1 之间的随机数
这里还是建议把注释放在后面

@KassieZ KassieZ added the has comments Some comments not resolved yet label Jan 24, 2025
@xiaozhuoyang
modify random and pmod
7d0cfb9 
Signed-off-by: leozhuo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@morrySnow morrySnow Awaiting requested review from morrySnow

@wangtianyi2004 wangtianyi2004 Awaiting requested review from wangtianyi2004

Assignees
No one assigned
Labels
has comments Some comments not resolved yet request review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@xiaozhuoyang @wangtianyi2004 @KassieZ