rohityadavcloud commented on a change in pull request #5879:
URL: https://github.com/apache/cloudstack/pull/5879#discussion_r789561622
##########
File path: server/src/main/java/com/cloud/user/AccountManagerImpl.java
##########
@@ -358,10 +391,46 @@ public UserVO getSystemUser() {
@Override
public boolean start() {
+ if (apiNameList == null) {
+ long startTime = System.nanoTime();
+ apiNameList = new ArrayList<String>();
+ Set<Class<?>> cmdClasses = new LinkedHashSet<Class<?>>();
+ for (PluggableService service : services) {
+ s_logger.debug(String.format("getting api commands of service:
%s", service.getClass().getName()));
+ cmdClasses.addAll(service.getCommands());
+ }
+ apiNameList = createApiNameList(cmdClasses);
+ long endTime = System.nanoTime();
+ s_logger.info("Api Discovery Service: Annotation, docstrings, api
relation graph processed in " + (endTime - startTime) / 1000000.0 + " ms");
+ }
_executor.scheduleAtFixedRate(new AccountCleanupTask(),
_cleanupInterval, _cleanupInterval, TimeUnit.SECONDS);
return true;
}
+ protected List<String> createApiNameList(Set<Class<?>> cmdClasses) {
+ List<String> apiNameList = new ArrayList<String>();
+
+ for (Class<?> cmdClass : cmdClasses) {
Review comment:
Can we re-use parts of this from api discovery manager/service?
--
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]
