-
Notifications
You must be signed in to change notification settings - Fork 19
Describe Delegated Storage concept #583
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
Conversation
5d33c4d to
3d7a81c
Compare
3d7a81c to
b02ed87
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.
Overall, I agree with this proposal, it was easy to ready and follow 👍
IMO as a follow-up after this proposal, we start with investigations to check how we can reuse as much as we can from available projects/tools.
E.g. do an investigation if we can easily wrap the Dapr store management into this concept. I see a lot of correlation.
Other idea is to, decided that we will do the minimal scope, and all of those will be implemented with Go libs e.g. https://github.com/graymeta/stow, https://github.com/google/go-cloud just as w minimal scope. We will prove that it’s working. Next, we will extract it to dedicated services and use gRPC. Unfortunately, currently, I'm not convinced that we need the whole concept to be implemented. Of course, we need to describe the target arch, but maybe it will be beneficial to describe also midterm solution.
9927db8 to
80eea44
Compare
c7ccf0e to
2460f3a
Compare
a5e56fc to
623f2be
Compare
mszostok
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.
Still a few things need to be described, but as before I agree with the overall proposal 👍
06496e4 to
db5fc3b
Compare
mszostok
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.
79a75bb to
f8359be
Compare

Description
Changes proposed in this pull request: