You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@drill.apache.org by vo...@apache.org on 2018/08/18 14:03:46 UTC

[drill] 01/03: DRILL-6689: Include query user information to drillbit.log

This is an automated email from the ASF dual-hosted git repository.

volodymyr pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/drill.git

commit 0000641800653ba3f41c6ef82a822a4e776d7b59
Author: Bohdan Kazydub <bo...@gmail.com>
AuthorDate: Wed Aug 15 14:06:54 2018 +0300

    DRILL-6689: Include query user information to drillbit.log
    
    closes #1432
---
 .../src/main/java/org/apache/drill/exec/work/foreman/Foreman.java    | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java b/exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java
index 16e8eb0..fa085e9 100644
--- a/exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java
+++ b/exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java
@@ -260,9 +260,10 @@ public class Foreman implements Runnable {
         break;
       case SQL:
         final String sql = queryRequest.getPlan();
-        // log query id and query text before starting any real work. Also, put
+        // log query id, username and query text before starting any real work. Also, put
         // them together such that it is easy to search based on query id
-        logger.info("Query text for query id {}: {}", this.queryIdString, sql);
+        logger.info("Query text for query with id {} issued by {}: {}", queryIdString,
+            queryContext.getQueryUserName(), sql);
         runSQL(sql);
         break;
       case EXECUTION: