You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by si...@apache.org on 2014/12/06 07:06:33 UTC
bookkeeper git commit: BOOKKEEPER-809: Wrong metric on LedgerDeleteOp
and LedgerOpenOp (Charles Xie via sijie)
Repository: bookkeeper
Updated Branches:
refs/heads/master 130ebfa8f -> 0dfb4fb94
BOOKKEEPER-809: Wrong metric on LedgerDeleteOp and LedgerOpenOp (Charles Xie via sijie)
Project: http://git-wip-us.apache.org/repos/asf/bookkeeper/repo
Commit: http://git-wip-us.apache.org/repos/asf/bookkeeper/commit/0dfb4fb9
Tree: http://git-wip-us.apache.org/repos/asf/bookkeeper/tree/0dfb4fb9
Diff: http://git-wip-us.apache.org/repos/asf/bookkeeper/diff/0dfb4fb9
Branch: refs/heads/master
Commit: 0dfb4fb94d0ccc98e1050f662712d85bcf2003b1
Parents: 130ebfa
Author: Sijie Guo <si...@apache.org>
Authored: Fri Dec 5 22:05:19 2014 -0800
Committer: Sijie Guo <si...@apache.org>
Committed: Fri Dec 5 22:05:19 2014 -0800
----------------------------------------------------------------------
.../main/java/org/apache/bookkeeper/client/LedgerDeleteOp.java | 4 ++--
.../src/main/java/org/apache/bookkeeper/client/LedgerOpenOp.java | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/bookkeeper/blob/0dfb4fb9/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerDeleteOp.java
----------------------------------------------------------------------
diff --git a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerDeleteOp.java b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerDeleteOp.java
index f8caa5b..b41d15a 100644
--- a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerDeleteOp.java
+++ b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerDeleteOp.java
@@ -81,9 +81,9 @@ class LedgerDeleteOp extends OrderedSafeGenericCallback<Void> {
@Override
public void safeOperationComplete(int rc, Void result) {
if (BKException.Code.OK != rc) {
- deleteOpLogger.registerFailedEvent(startTime);
+ deleteOpLogger.registerFailedEvent(MathUtils.elapsedMSec(startTime));
} else {
- deleteOpLogger.registerSuccessfulEvent(startTime);
+ deleteOpLogger.registerSuccessfulEvent(MathUtils.elapsedMSec(startTime));
}
cb.deleteComplete(rc, this.ctx);
}
http://git-wip-us.apache.org/repos/asf/bookkeeper/blob/0dfb4fb9/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerOpenOp.java
----------------------------------------------------------------------
diff --git a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerOpenOp.java b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerOpenOp.java
index 4df89ea..eb145bd 100644
--- a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerOpenOp.java
+++ b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerOpenOp.java
@@ -196,9 +196,9 @@ class LedgerOpenOp implements GenericCallback<LedgerMetadata> {
void openComplete(int rc, LedgerHandle lh) {
if (BKException.Code.OK != rc) {
- openOpLogger.registerFailedEvent(startTime);
+ openOpLogger.registerFailedEvent(MathUtils.elapsedMSec(startTime));
} else {
- openOpLogger.registerSuccessfulEvent(startTime);
+ openOpLogger.registerSuccessfulEvent(MathUtils.elapsedMSec(startTime));
}
cb.openComplete(rc, lh, ctx);
}