smengcl commented on code in PR #9313:
URL: https://github.com/apache/ozone/pull/9313#discussion_r2545952359
##########
hadoop-ozone/interface-client/src/main/proto/OmClientProtocol.proto:
##########
@@ -881,7 +881,7 @@ message SnapshotInfo {
optional hadoop.hdds.UUID globalPreviousSnapshotID = 9;
optional string snapshotPath = 10;
optional string checkpointDir = 11 [deprecated = true];
- optional int64 dbTxSequenceNumber = 12;
+ optional int64 dbTxSequenceNumber = 12 [deprecated = true];
Review Comment:
It might be safe to just remove this field later, so that when SnapshotInfo
is serialized again this field will be ignored while maintaining compatibility,
as long as we don't use field number `12` ever again for this message.
--
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]
