rhtyd commented on a change in pull request #4121:
URL: https://github.com/apache/cloudstack/pull/4121#discussion_r433774346
##########
File path:
framework/db/src/main/java/com/cloud/utils/db/TransactionMBeanImpl.java
##########
@@ -45,14 +48,15 @@ public void removeTransaction(TransactionLegacy txn) {
@Override
public int getTransactionCount() {
- return _txns.size();
+ return totalTransactionCount.intValue();
}
@Override
public int[] getActiveTransactionCount() {
- int[] count = new int[2];
+ int[] count = new int[3];
count[0] = 0;
count[1] = 0;
+ count[2] = _txns.size();
Review comment:
Removed this, not transactions would return the current size; instead of
cumulative/historic size.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
