You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by ta...@apache.org on 2019/04/16 20:24:56 UTC

[impala] 05/05: Log scanner Open() errors

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

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

commit ded73049673eb44bb02ad3f7277d1e7915998bc0
Author: Bharath Vissapragada <bh...@cloudera.com>
AuthorDate: Mon Apr 15 13:27:53 2019 -0700

    Log scanner Open() errors
    
    While the status is propagated to the runtime profile, it helps
    to have it in the log file in cases where the profile is not
    available.
    
    Testing: Verified the message is logged by setting PREPARE_SCANNER
    debug action.
    
    Change-Id: I595d509ca24077b924923fa9aa4cdb912c21c9f2
    Reviewed-on: http://gerrit.cloudera.org:8080/13018
    Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 be/src/exec/hdfs-scan-node.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/be/src/exec/hdfs-scan-node.cc b/be/src/exec/hdfs-scan-node.cc
index 66b7a8b..545d88d 100644
--- a/be/src/exec/hdfs-scan-node.cc
+++ b/be/src/exec/hdfs-scan-node.cc
@@ -511,8 +511,8 @@ void HdfsScanNode::ProcessSplit(const vector<FilterContext>& filter_ctxs,
     if (VLOG_QUERY_IS_ON) {
       stringstream ss;
       ss << "Error preparing scanner for scan range " << scan_range->file() <<
-          "(" << scan_range->offset() << ":" << scan_range->len() << ").";
-      ss << endl << runtime_state_->ErrorLog();
+          "(" << scan_range->offset() << ":" << scan_range->len() << "). ";
+      ss << status.msg().msg() << endl << runtime_state_->ErrorLog();
       VLOG_QUERY << ss.str();
     }