You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by cs...@apache.org on 2019/06/27 11:52:47 UTC
[impala] 03/07: Fix incorrect timestamp units in the logs
This is an automated email from the ASF dual-hosted git repository.
csringhofer pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git
commit 5d42fa3ad6aa5778a2fbab3caa8091985718d910
Author: Bharath Vissapragada <bh...@cloudera.com>
AuthorDate: Tue Jun 25 16:12:54 2019 -0700
Fix incorrect timestamp units in the logs
EventSequence.markEvent() returns ns, so the existing conversion
is incorrect.
Change-Id: I132a007404dcf5291e70f2642ed3e9f995d694b1
Reviewed-on: http://gerrit.cloudera.org:8080/13731
Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
.../java/org/apache/impala/authorization/BaseAuthorizationChecker.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fe/src/main/java/org/apache/impala/authorization/BaseAuthorizationChecker.java b/fe/src/main/java/org/apache/impala/authorization/BaseAuthorizationChecker.java
index 2104ed7..15cbb9a 100644
--- a/fe/src/main/java/org/apache/impala/authorization/BaseAuthorizationChecker.java
+++ b/fe/src/main/java/org/apache/impala/authorization/BaseAuthorizationChecker.java
@@ -92,7 +92,7 @@ public abstract class BaseAuthorizationChecker implements AuthorizationChecker {
if (authzCtx.getTimeline().isPresent()) {
EventSequence timeline = authzCtx.getTimeline().get();
long durationMs = timeline.markEvent(String.format("Authorization finished (%s)",
- config_.getProviderName())) / 1000;
+ config_.getProviderName())) / 1000000;
LOG.debug("Authorization check took {} ms", durationMs);
}
}