DaanHoogland commented on a change in pull request #5879:
URL: https://github.com/apache/cloudstack/pull/5879#discussion_r792431977
##########
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:
I'll look into it again. It didn't seem prudent when I implemented this.
--
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]
