-
Notifications
You must be signed in to change notification settings - Fork 587
HDDS-8214. Recon - OM DB Insights - Key Level Info #4516
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
Merged
Merged
Changes from 18 commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
bb3704c
HDDS-8214. Recon - OM DB Insights - Container Level Info.
823ead5
HDDS-8214. Recon - OM DB Insights - Key Level Info.
f8e0d79
HDDS-8214. Recon - OM DB Insights - Key Level Info.
8dacd12
HDDS-8214. Recon - OM DB Insights - Key Level Info.
168e651
HDDS-8214. Recon - OM DB Insights - Key Level Info.
cace42c
HDDS-8214. Recon - OM DB Insights - Key Level Info.
f628968
HDDS-8214. Recon - OM DB Insights - Key Level Info.
9fc1eaf
HDDS-8214. Recon - OM DB Insights - Key Level Info.
285ddb1
HDDS-8214. Recon - OM DB Insights - Key Level Info.
05141a3
HDDS-8214. Recon - OM DB Insights - Key Level Info.
7091198
HDDS-8214. Recon - OM DB Insights - Key Level Info.
fb210c3
HDDS-8214. Recon - OM DB Insights - Key Level Info.
eb23a87
HDDS-8214. Recon - OM DB Insights - Key Level Info.
3e1e87b
HDDS-8214. Recon - OM DB Insights - Key Level Info.
72f6c40
HDDS-8214. Recon - OM DB Insights - Key Level Info.
18abcb9
HDDS-8214. Recon - OM DB Insights - Key Level Info.
410b846
HDDS-8214. Recon - OM DB Insights - Key Level Info.
737734c
HDDS-8214. Recon - OM DB Insights - Key Level Info.
25171b5
HDDS-8214. Recon - OM DB Insights - Key Level Info.
a169db1
HDDS-8214. Recon - OM DB Insights - Key Level Info.
90c5a15
HDDS-8214. Recon - OM DB Insights - Key Level Info.
6135735
HDDS-8214. Recon - OM DB Insights - Key Level Info.
5ae1ee4
HDDS-8214. Recon - OM DB Insights - Key Level Info.
76dd31a
HDDS-8214. Recon - OM DB Insights - Key Level Info.
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Unable to get use case where getting keys only for deleted container from SCM. I think we should return all keys which is not present in SCM (including deleted key) which may have data loss.
Additionally need see if gap between SCM and DN if DN do not have data.
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.
As discussed, Please conclude on the use case.