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

[typo](doc)Update materialized-view.md #18826

Merged
merged 2 commits into from
Apr 21, 2023
Merged

[typo](doc)Update materialized-view.md #18826

merged 2 commits into from
Apr 21, 2023

Conversation

lljqy
Copy link
Contributor

@lljqy lljqy commented Apr 19, 2023

文档中“取消物化视图”的语法有误,少了个“TABLE”关键字

Proposed changes

Issue Number: close #18825

Problem summary

Describe your changes.

Checklist(Required)

  • Does it affect the original behavior
  • Has unit tests been added
  • Has document been added or modified
  • Does it need to update dependencies
  • Is this PR support rollback (If NO, please explain WHY)

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...

文档中“取消物化视图”的语法有误,少了个“TABLE”关键字
@github-actions github-actions bot added the kind/docs Categorizes issue or PR as related to documentation. label Apr 19, 2023
luzhijing
luzhijing previously approved these changes Apr 19, 2023
Copy link
Contributor

@luzhijing luzhijing left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for your Pull Request and Welcome to Apache Doris Community.

@luzhijing luzhijing changed the title Update materialized-view.md [typo](doc)Update materialized-view.md Apr 19, 2023
@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Apr 19, 2023
@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@luzhijing
Copy link
Contributor

run buildall

Co-authored-by: Pxl <pxl290@qq.com>
Copy link
Contributor

@luzhijing luzhijing left a comment

Choose a reason for hiding this comment

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

LGTM

@luzhijing
Copy link
Contributor

run buildall

1 similar comment
@luzhijing
Copy link
Contributor

run buildall

@luzhijing luzhijing merged commit 5a0c1cb into apache:master Apr 21, 2023
gnehil pushed a commit to gnehil/doris that referenced this pull request Apr 21, 2023
Reminiscent pushed a commit to Reminiscent/doris that referenced this pull request May 15, 2023
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. kind/docs Categorizes issue or PR as related to documentation. reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

doris官方文档有错误
3 participants