-
Notifications
You must be signed in to change notification settings - Fork 4
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
Handle complex metadata permission storage #58
base: main
Are you sure you want to change the base?
Conversation
The permissions metadata tracker should:
The DSI documentation for back-end drivers should be extended to explain this behavior. In a maximally conservative workflow, we would never save data, but always read data using plugins before incorporating query functionality. Coordinate with @kchilleri on this, and feel free to create sub-issues as needed. |
Updated this branch to catch up with main and to implement the functionalities requested. Some implementation notes:
No rush to review, holidays are coming up! :) |
@qwofford @kchilleri I can't add y'all as reviewers since Quincy is the owner or the PR and Krishna isn't listed, but see above ^ |
Note that we should squash these commits because I made a mess of the git history while resolving conflicts. |
Looking excellent as usual. I'm adding some documentation for this new capability. When you @DanielRJohnson have an opportunity please take a look at what I wrote and we can discuss after the holiday: https://lanl.github.io/dsi/permissions.html |
See #39