-
Notifications
You must be signed in to change notification settings - Fork 20
set permission for kvs file #107
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
base: main
Are you sure you want to change the base?
set permission for kvs file #107
Conversation
e60d583 to
3e2d2e2
Compare
License Check Results🚀 The license check job ran with the Bazel command: bazel run //:license-checkStatus: Click to expand output |
- set permission for kvs file Fixes eclipse-score#99
3e2d2e2 to
b1426c9
Compare
|
The created documentation from the pull request is available at: docu-html |
|
Looks good. @vinodreddy-g Will the api change for the builder ("storage_mode" method) be implemented in Rust? |
|
@joshualicht @Ahmed-Elsaka-JC We can discuss in the coming meeting also . We could have the permissions for KVS , but we should not expose the os abstraction enum to the api level ,Since we might use a different file storage api or may be KVS is not file based for longterm. We could have a KVS enum for it and map it tot os abstraction internally. |
|
Okay I put it on the meeting-board👍🏻 |
Closes #99