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.

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

[docs]Fix statement docs of CANCAL, REFRESH, PAUSE and RESUME MATERIALIZED VIEW #1902

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lsy3993
Copy link
Contributor

@lsy3993 lsy3993 commented Jan 22, 2025

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 22, 2025 09:27
```

## Required Parameters
**<task_id>**
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. 参数要一个序号
  2. 需要加上反引号
Suggested change
**<task_id>**
**1. `<task_id>`**

Where:
```sql
refresh_type
: [partitionSpec | COMPLETE | AUTO]
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
: [partitionSpec | COMPLETE | AUTO]
: [ <partitionSpec> | COMPLETE | AUTO ]


```sql
partitionSpec
: PARTITIONS (<partition_name1>[ , partition_name2 ][ ... ])
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
: PARTITIONS (<partition_name1>[ , partition_name2 ][ ... ])
: PARTITIONS (<partition_name> [, <partition_name> [, ... ] ])

**<refresh_type>**
> Specifies the refresh type of this materialized view.
>
> The refresh type may be one of the partitionSpec, COMPLETE or AUTO.
Copy link
Contributor

Choose a reason for hiding this comment

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

缺少可选参数<partition_name>的说明

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants