swamirishi commented on code in PR #7193:
URL: https://github.com/apache/ozone/pull/7193#discussion_r1767776952
##########
hadoop-ozone/interface-client/src/main/proto/OmClientProtocol.proto:
##########
@@ -1379,6 +1379,8 @@ message PurgeKeysRequest {
// if set, will purge keys in a snapshot DB instead of active DB
optional string snapshotTableKey = 2;
repeated SnapshotMoveKeyInfos keysToUpdate = 3;
+ // previous snapshotID can also be null & this field would be absent in
older requests.
+ optional NullableUUID expectedPreviousSnapshotID = 4;
Review Comment:
Since there is no direct explicit way to differentiate b/w older requests &
null values. I had to create a wrapper which means I can set NullableUUID which
doesn't have anything inside. Since uuid field inside NullableUUID type is
optional, we can signify this as a newer request and having nothing inside the
field would signify a null value.
--
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]
