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
HDDS-2180. Add Object ID and update ID on VolumeList Object. #1526
Conversation
@@ -95,13 +100,18 @@ protected VolumeList addVolumeToOwnerList(VolumeList volumeList, | |||
} | |||
|
|||
List<String> prevVolList = new ArrayList<>(); | |||
long objectID = 100; |
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.
Is there a reason for initialize objectID to 100 here for new volume?
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.
No, thanks I will fix that, I discovered it from a test code path, and left over from the fix. I will get this fixed.
...e/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/volume/OMVolumeRequest.java
Show resolved
Hide resolved
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.
LGTM, some minor questions inline...
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
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.
+1 pending CI.
Was this committed without a pre-commit run from Anzix? |
Looks like it was not run. I did look at the yetus output but the output is too noisy to make any sense. Is this commit causing any issues ? |
None that I know of. However we should not commit a patch without CI on principle. |
This has been broken for quite a while; So I rely on Jenkins/Yetus and hand building before apply the commits. |
https://issues.apache.org/jira/browse/HDDS-2180
Adds Object ID and Update ID to VolumeList object.