sumitagrawl commented on code in PR #7647:
URL: https://github.com/apache/ozone/pull/7647#discussion_r1925049236
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/file/OMDirectoryCreateRequest.java:
##########
@@ -121,6 +122,18 @@ public OMRequest preExecute(OzoneManager ozoneManager)
throws IOException {
}
+ public OmLockOpr.OmLockInfo lock(OzoneManager ozoneManager, OmLockOpr
lockOpr) throws IOException {
Review Comment:
@errose28 We have, `OMDirectoryCreateRequest for OBS` and
OMDirectoryCreateRequestWithFSO for FSO.
So above class is OBS only (not FSO)
Legacy many places is treated as OBS only, so from this code perspective, it
can not distinguish if LEGACY or OBS, so precautionary added the check for OBS.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
