-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Package management bookkeeper storage implementation #8744
Package management bookkeeper storage implementation #8744
Conversation
70887c2
to
baa84b1
Compare
baa84b1
to
815ffe6
Compare
@sijie @jiazhai @codelipenghui Please take a look when you have time. Thanks. |
ping |
ping @codelipenghui @jiazhai @sijie |
ping |
|
||
@Override | ||
public CompletableFuture<Void> closeAsync() { | ||
return CompletableFuture.runAsync(() -> this.namespace.close()); |
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.
We should also close the DistributedLogManager
?
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.
Master Issue: apache#8676 ### Motivation Support bookkeeper storage for packages management service. ### Modifications - Add bookkeeper storage provider - Add bookkeeper storage unit tests
Master Issue: #8676
Motivation
Support bookkeeper storage for packages management service.
Modifications