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


##########
hadoop-hdds/interface-client/src/main/proto/hdds.proto:
##########
@@ -408,6 +408,11 @@ message BlockID {
     optional uint64 blockCommitSequenceId = 2 [default = 0];
 }
 
+message DeletedBlock {

Review Comment:
   The DeletedBlock class is used by both the client and server, so it should 
reside in the hdds-common module and must have access to the corresponding 
protobuf definitions. Since ScmServerProtocol.proto is not accessible from the 
org.apache.hadoop.hdds.client package, we need to define the proto in 
hdds.proto instead.



-- 
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: issues-unsubscr...@ozone.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org
For additional commands, e-mail: issues-h...@ozone.apache.org

Reply via email to