[impala] 03/07: Fix incorrect timestamp units in the logs

[cs...@apache.org]

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);
     }
   }