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

Fix routine load statements documentation docs #1915

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

Conversation

liujiwen-up
Copy link
Contributor

Versions

  • dev
  • 3.0
  • 2.1
  • 2.0

Languages

  • Chinese
  • English

Docs Checklist

  • Checked by AI
  • Test Cases Built

Copy link
Contributor

@morrySnow morrySnow left a comment

Choose a reason for hiding this comment

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

一些通用的问题。评论到了第一次发现的地方,其他文档也检查修改一下吧

[job_properties]
FROM data_source
[data_source_properties]
ALTER ROUTINE LOAD FOR [<db.>]<job_name>
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
ALTER ROUTINE LOAD FOR [<db.>]<job_name>
ALTER ROUTINE LOAD FOR [<db>.]<job_name>


## Required Parameters

**1. `[db.]job_name`**
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
**1. `[db.]job_name`**
**1. `[<db>.]<job_name>`**

>
> The identifier cannot use reserved keywords. For more details, please refer to identifier requirements and reserved keywords.

**2. `job_properties`**
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
**2. `job_properties`**
**2. `<job_properties>`**

[job_properties]
FROM data_source [data_source_properties]
[COMMENT "comment"]
CREATE ROUTINE LOAD [<db.>]<job_name> [ON <tbl_name>]
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
CREATE ROUTINE LOAD [<db.>]<job_name> [ON <tbl_name>]
CREATE ROUTINE LOAD [<db>.]<job_name> [ON <tbl_name>]

[<load_properties>]
[<job_properties>]
FROM <data_source> [<data_source_properties>]
[<COMMENT "comment">]
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
[<COMMENT "comment">]
[COMMENT "<comment>"]

@@ -25,31 +25,51 @@ under the License.
-->


## Description
## Description## Description
Copy link
Contributor

Choose a reason for hiding this comment

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

这里粘贴错了?


```sql
PAUSE [ALL] ROUTINE LOAD FOR job_name
PAUSE [<ALL>] ROUTINE LOAD FOR <job_name>
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
PAUSE [<ALL>] ROUTINE LOAD FOR <job_name>
PAUSE [ALL] ROUTINE LOAD FOR <job_name>

## Example
## Required Parameters

**1. `job_name`**
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
**1. `job_name`**
**1. `<job_name>`**

@KassieZ KassieZ added has comments Some comments not resolved yet and removed request review labels Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has comments Some comments not resolved yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants