-
Notifications
You must be signed in to change notification settings - Fork 462
[lake] Introduce tiering assignment manager to CoordinatorServer #780
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
[lake] Introduce tiering assignment manager to CoordinatorServer #780
Conversation
98dd54d to
41a2e6b
Compare
...s-server/src/test/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManagerTest.java
Outdated
Show resolved
Hide resolved
...s-server/src/test/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManagerTest.java
Outdated
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
| private class ExpiredOperationReaper extends ShutdownableThread { | ||
|
|
||
| public ExpiredOperationReaper() { | ||
| super("ExpiredOperationReaper", false); |
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.
"LakeTieringExpiredOperationReaper" to distinguish with the name of DelayedOperationManager.ExpiredOperationReaper.
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
fb6432a to
f27ea79
Compare
|
@wuchong Comments addressed. |
f27ea79 to
1ef3c0a
Compare
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Show resolved
Hide resolved
fluss-server/src/main/java/com/alibaba/fluss/server/coordinator/LakeTableTieringManager.java
Outdated
Show resolved
Hide resolved
813807b to
42ee3e9
Compare
|
@wuchong Comments addressed. |
42ee3e9 to
8bedfab
Compare
c4dd395 to
8450590
Compare
8450590 to
a387802
Compare
|
@luoyuxia please take a look the modified changes. |
luoyuxia
left a comment
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.
@wuchong Thanks for the suggestion commits, LGTM!
| * Tiering --> |lake tiering report tiering| Failed | ||
| * Tiered -> Scheduled | ||
| * Failed -> Pending | ||
| * ┌─────┐ ┌──────┐ |
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.
This digram looks cool!
…che#780) --------- Co-authored-by: Jark Wu <[email protected]>
…che#780) --------- Co-authored-by: Jark Wu <[email protected]>
…che#780) --------- Co-authored-by: Jark Wu <[email protected]>
Purpose
Linked issue: close #433
Brief change log
table.datalake.tiering.intervalto control the data fressness tiered from fluss in datalake.LakeTableTIeringManagerto coordinator server to manage the lifecycle of the lake table to be tiered, the state macheine looks as follows:Then, LakeTiering can requet table from
LakeTableTIeringManager:1: After requet a table, it wil send heartbeat periodically to tell coordiantor the table is still tiering.
2: If the table hasn't been notified to be alive for a while, coordinator will assign the table to another tiering service
3: After tiering service finish one round of tiering for the table, notify the table has finished the tiering via heartbeat. Then
LakeTableTIeringManagerwill waittable.datalake.tiering.intervalto assign the table to other tiering serviceTests
LakeTableTieringManagerTest.
Let's add IT after we introduce RPC in 432
API and Format
Documentation