[GitHub] [ignite] Vladsz83 commented on a diff in pull request #10783: IGNITE-19766 Added topology snapshot info to services system view

["Vladsz83 (via GitHub)" <gi...@apache.org>]

Vladsz83 commented on code in PR #10783:
URL: https://github.com/apache/ignite/pull/10783#discussion_r1236641674


##########
modules/core/src/main/java/org/apache/ignite/spi/systemview/view/ServiceView.java:
##########
@@ -98,4 +99,14 @@ public boolean staticallyConfigured() {
     public UUID originNodeId() {
         return serviceInfo.originNodeId();
     }
+
+    /**
+     * Gets service deployment topology snapshot. Service topology snapshot is represented

Review Comment:
   Other descriptions of the properties  are short int this class. Maybe just '@return Map of number of service instances per node ID.' and '@see ServiceDescriptor'? WDYT?



-- 
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: notifications-unsubscribe@ignite.apache.org

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