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 <[email protected]> Authored: Fri Dec 5 22:05:19 2014 -0800 Committer: Sijie Guo <[email protected]> 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); }
