-
Notifications
You must be signed in to change notification settings - Fork 462
Add document about versioned merge engine #523
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
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -5,4 +5,66 @@ sidebar_position: 3 | |||||
|
|
||||||
| # Versioned Merge Engine | ||||||
|
|
||||||
| TODO: Fill me #459 | ||||||
| By specifying `'table.merge-engine' = 'versioned'`, users can update data based on the configured version column. Updates will be carried out when the latest value of the specified field is greater than the stored value. If it is less than or empty, no update will be made. | ||||||
|
|
||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'd like to suggest to add a statement to explain how the feature is useful just like |
||||||
| :::note | ||||||
| When using `versioned` merge engine, there are the following limits: | ||||||
| - `UPDATE` and `DELETE` statements are not supported | ||||||
| - Partial update is not supported | ||||||
|
||||||
| ::: | ||||||
|
||||||
| ::: | |
| ::: |
Outdated
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.
| ::: | |
| ::: |
Outdated
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.
| ```sql title="versioned" | |
| ```sql title="Flink SQL" |
Outdated
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.
Could you please change SQL keyword to uppercase to align the style of first-row merge engine page?
Outdated
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.
nit:
| insert into merge_engine_with_version ( a,b,ts ) VALUES (1, 'v1', 1000); | |
| insert into merge_engine_with_version (a, b, ts ) VALUES (1, 'v1', 1000); |
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.
@luoyuxia Whether to remove the space after ts?
(a, b, ts ) -> (a, b, ts)
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.
Haven't noticed that
Outdated
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.
nit:
| insert into merge_engine_with_version ( a,b,ts ) VALUES (1, 'v2', 999); | |
| insert into merge_engine_with_version (a, b, ts ) VALUES (1, 'v2', 999); |
Outdated
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.
nit
| insert into merge_engine_with_version ( a,b,ts ) VALUES (1, 'v2', 2000); | |
| insert into merge_engine_with_version (a, b, ts ) VALUES (1, 'v2', 2000); |
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.
Maybe add -- +---+-----+------+ in table end.
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.
nit: add the bottom line to align with the style of above content
-- +---+-----+------+
-- | a | b | ts |
-- +---+-----+------+
-- | 1 | v2 | 2000 |
-- +---+-----+------+
Outdated
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.
typo: nsert -> insert.
Outdated
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.
| nsert into merge_engine_with_version ( a,b,ts ) VALUES (1, 'v2', null); | |
| insert into merge_engine_with_version (a, b, ts ) VALUES (1, 'v2', null); |
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.
same as above.
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.
dito
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.
nit: