DaanHoogland commented on a change in pull request #5588:
URL: https://github.com/apache/cloudstack/pull/5588#discussion_r782311347
##########
File path: engine/schema/src/main/java/com/cloud/host/dao/HostDao.java
##########
@@ -142,4 +142,11 @@
HostVO findByName(String name);
List<HostVO> listHostsWithActiveVMs(long offeringId);
+
+ /**
+ * get the number of hosts/agents this {@see ManagementServer} has
responsibility over
+ * @param msid the id of the {@see ManagementServer}
+ * @return the number of hosts/agents this {@see ManagementServer} has
responsibility over
+ */
+ int countByMs(long msid);
Review comment:
I've taken over this name from somewhere and have no preference, also
the lowercase `msid` is on the API as opposed to the camelcase `msId` so I'll
leave as is.
--
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]
