sureshanaparti commented on code in PR #18:
URL: https://github.com/apache/cloudstack-csbench/pull/18#discussion_r2120111754


##########
apirunner/apirunner.go:
##########
@@ -124,11 +125,16 @@ func executeAPIandCalculate(profileName string, apiURL 
string, command string, p
        var avgTime float64
        var totalTime float64
        var count float64
+       getRequestList := 
map[string]struct{}{"isaccountallowedtocreateofferingswithtags": {}, 
"readyforshutdown": {}, "cloudianisenabled": {}, "quotabalance": {}, 
+       "quotasummary": {}, "quotatarifflist": {}, "quotaisenabled": {}, 
"quotastatement": {}, "verifyoauthcodeandgetuser": {}}
+       _, isInGetRequestList := getRequestList[command]
+       isGetRequest, _ := regexp.MatchString("^(get|list|query|find)(\\w+)+$", 
command)
+
        if iterations != 1 {
                log.Infof("Calling API %s for %d number of iterations with 
parameters %s", command, iterations, params)
                for i := 1; i <= iterations; i++ {
                        log.Infof("Started with iteration %d for the command 
%s", i, command)
-                       elapsedTime, apicount, result := executeAPI(apiURL, 
params)
+                       elapsedTime, apicount, result := executeAPI(apiURL, 
params, isGetRequest || isInGetRequestList)

Review Comment:
   ```suggestion
                        elapsedTime, apicount, result := executeAPI(apiURL, 
params, !(isGetRequest || isInGetRequestList))
   ```



-- 
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: dev-unsubscr...@cloudstack.apache.org

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

Reply via email to