hanishakoneru commented on a change in pull request #877: HDDS-1618. Merge code
for HA and Non-HA OM write type requests for bucket
URL: https://github.com/apache/hadoop/pull/877#discussion_r290492749
##########
File path:
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/response/OMVolumeCreateResponse.java
##########
@@ -66,5 +66,16 @@ public OmVolumeArgs getOmVolumeArgs() {
return omVolumeArgs;
}
+ @Override
+ public void addResponseToOMDB(OMMetadataManager omMetadataManager)
+ throws IOException {
+ String dbVolumeKey =
+ omMetadataManager.getVolumeKey(omVolumeArgs.getVolume());
+ String dbUserKey =
+ omMetadataManager.getUserKey(omVolumeArgs.getOwnerName());
+
+ omMetadataManager.getVolumeTable().put(dbVolumeKey, omVolumeArgs);
+ omMetadataManager.getUserTable().put(dbUserKey, volumeList);
Review comment:
Does the volumeList include the previous volumes owned by the user or just
the current volume?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
