Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18682 )
Change subject: IMPALA-11406: Fix incorrect duration log for authorization ...................................................................... IMPALA-11406: Fix incorrect duration log for authorization IMPALA-8443 extends EventSequence.markEvent() to return the duration between the last and the current event. However, the duration is calculated using the start time, not the last time it's invoked, which causes misleading time in logs of "Authorization check took n ms". This fixes the bug and also adds a log for the analysis duration. Change-Id: I8b665f1b4ac86577711598ce9d845cf82fedbcd7 Reviewed-on: http://gerrit.cloudera.org:8080/18682 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/analysis/AnalysisContext.java M fe/src/main/java/org/apache/impala/util/EventSequence.java 2 files changed, 8 insertions(+), 3 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/18682 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8b665f1b4ac86577711598ce9d845cf82fedbcd7 Gerrit-Change-Number: 18682 Gerrit-PatchSet: 4 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Tamas Mate <[email protected]>
