[GitHub] [incubator-uniffle] advancedxy commented on a diff in pull request #449: [Feature] shuffle server report storage capability

[GitBox <gi...@apache.org>]

advancedxy commented on code in PR #449:
URL: https://github.com/apache/incubator-uniffle/pull/449#discussion_r1058689270


##########
proto/src/main/proto/Rss.proto:
##########
@@ -239,6 +239,7 @@ message ShuffleServerHeartBeatRequest {
   int32 eventNumInFlush = 5;
   repeated string tags = 6;
   google.protobuf.BoolValue isHealthy = 7;
+  map<string, LocalStorageInfo> localStorages = 21; // mount point to local storage info mapping.

Review Comment:
   What properties do you want to have for RemoteStorageInfo? I thought about that when implementing this PR, but didn't find anything meaning and general enough. If you have some ideas/suggestions, I'd appreciate that.
   
   



-- 
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-unsubscribe@uniffle.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@uniffle.apache.org
For additional commands, e-mail: issues-help@uniffle.apache.org