-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
[FLINK-37087][doc] Add docs for alter materialized table as query #26064
base: master
Are you sure you want to change the base?
Conversation
3217e5f
to
9036d53
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see that the REFRESH keyword is mentioned - there is a some explanation of this in the file but not how the partition piece works and when you would want to use it.
I see we have added SUSPEND and RESUME, it would be useful to describe or link to how these work - including how the supplied list of keys should be used.
bc40df2
to
38fd3d0
Compare
38fd3d0
to
b2fdeb7
Compare
Reviewed by Chi on 23/01/2025 Go back to the submitter with review comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hackergin Thanks for your contribution, I left some comments.
docs/content.zh/docs/dev/table/materialized-table/statements.md
Outdated
Show resolved
Hide resolved
docs/content.zh/docs/dev/table/materialized-table/statements.md
Outdated
Show resolved
Hide resolved
docs/content.zh/docs/dev/table/materialized-table/statements.md
Outdated
Show resolved
Hide resolved
docs/content.zh/docs/dev/table/materialized-table/statements.md
Outdated
Show resolved
Hide resolved
docs/content.zh/docs/dev/table/materialized-table/statements.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Ron <[email protected]>
@davidradl This document is solely intended to explain how users can use it. If we need to introduce the underlying implementation to users, I think we should have a separate document to explain the overall process. |
a2554b4
to
acc279c
Compare
What is the purpose of the change
Add docs for alter materialized table as query
Brief change log
Verifying this change
This change only contains document change.
Does this pull request potentially affect one of the following parts:
@Public(Evolving)
: (no)Documentation