-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[core] Supports skipping specified actions during writing. #4884
base: master
Are you sure you want to change the base?
[core] Supports skipping specified actions during writing. #4884
Conversation
9d3fcef
to
ed49eb8
Compare
Thanks for @LinMingQiang prepare this pr. |
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.
Good job and left some comments.
7250389
to
b3867f8
Compare
+1 |
<td><h5>write.skip-actions</h5></td> | ||
<td style="word-wrap: break-word;">(none)</td> | ||
<td>String</td> | ||
<td>This parameter only works when write-only is false., You can specify which actions to skip during the write process.<br />1. 'partition-expire': Skip the action of partition expiration.<br />2. 'snapshot-expire': Skip the action of snapshot expiration.<br />3. 'create-tag': Skip automatic tag creation.<br />Both can be configured at the same time: 'partition-expire,snapshot-expire,create-tag'.</td> |
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 prefer to add a separate enable conf for each such post-commit behavior, with the default set to true.
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.
Thanks for review, If the parameters are separated, will it be more cumbersome to use ? For my needs, I need to skip partition-expire
& snapshot-expire
& create tag
but retain compact
, so I may need to configure three parameters to meet my needs.
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 you can add a write-actions
? Default is 'all'. The values can be:
snapshot-expire
, partition-expire
, compaction
, tag-automatic-creation
.
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.
Does this compact
mean full
or minor
or both, or should it be separated into two actions full_compact
and minor_compact
?
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 full or minor should be separate option.
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.
Ok, I will fix it.
b3867f8
to
99dcebd
Compare
0e09ce9
to
0c0573a
Compare
0c0573a
to
7cb026d
Compare
Purpose
There are several operations during data writing such as
partition-expire
,snapshot-expire
,create-tag
...Currently, these actions are either executed together or not executed at all in the framework.
The purpose of this PR is to supports skipping specified actions during writing.
For example : set
'write.skip-actions'='partition-expire'
, the partition expiration action will be skipped during data writing, butsnapshot-expire
/auto create tag
/full compact
will still execute normally.Linked issue: close #xxx
Tests
API and Format
Documentation