sumitagrawl commented on code in PR #8566:
URL: https://github.com/apache/ozone/pull/8566#discussion_r2160786708


##########
hadoop-hdds/interface-server/src/main/proto/ScmServerProtocol.proto:
##########
@@ -167,7 +167,13 @@ message AllocateScmBlockRequestProto {
  * multiple number of keys (and their blocks).
  */
 message DeleteScmKeyBlocksRequestProto {
-  repeated KeyBlocks keyBlocks = 1;
+
+  // Field kept for rolling upgrades.  It contains KeyBlocks without
+  // per-block size information.
+  repeated KeyBlocks keyBlocks = 1 [deprecated = true];

Review Comment:
   but compatibility need not be at higher level structure, it can be done at 
sub-layer also if no impact. Its just data how you are populating.



-- 
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]

Reply via email to