Skip to content

Conversation

@sodamnsure
Copy link
Contributor

add stop instructions

Proposed changes

Issue Number: close #xxx

Problem Summary:

Describe the overview of changes.

Checklist(Required)

  1. Does it affect the original behavior: (Yes/No/I Don't know)
  2. Has unit tests been added: (Yes/No/No Need)
  3. Has document been added or modified: (Yes/No/No Need)
  4. Does it need to update dependencies: (Yes/No)
  5. Are there any changes that cannot be rolled back: (Yes/No)

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

add stop instructions
@luzhijing
Copy link
Contributor

You can update the EN docs together.

add stop instructions
@luzhijing
Copy link
Contributor

LGTM

@sodamnsure sodamnsure changed the title Update routine-load-manual.md [docs](routine-load)Update routine-load-manual.md Feb 11, 2022
@EmmyMiao87 EmmyMiao87 added good first issue kind/docs Categorizes issue or PR as related to documentation. labels Feb 11, 2022
用户可以通过 `STOP/PAUSE/RESUME` 三个命令来控制作业的停止(FE会自动定期清理停止的ROUTINE LOAD),暂停和重启。可以通过 `HELP STOP ROUTINE LOAD;`, `HELP PAUSE ROUTINE LOAD;` 以及 `HELP RESUME ROUTINE LOAD;` 三个命令查看帮助和示例。
## 其他说明
Copy link
Contributor

Choose a reason for hiding this comment

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

关于 STOP状态的 routine laod job 会被定期清理的解释,建议在其他说明中单独新增一项单独解释。

@EmmyMiao87 EmmyMiao87 self-assigned this Feb 11, 2022
Add Chinese notes in other instructions to explain the stopped operation
Add English notes in other instructions to explain the stopped operation
Delete redundant Chinese comments
Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions
Copy link
Contributor

PR approved by at least one committer and no changes requested.

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Feb 13, 2022
@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@morningman morningman merged commit 8d7a0d9 into apache:master Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. good first issue kind/docs Categorizes issue or PR as related to documentation. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants