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/02/16 01:20:58 UTC

[impala] 08/11: IMPALA-6942: Reword error message to say "Failed" rather than "Cancelled"

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

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

commit b1f43e41d9a9f59fe9f9bdef0c736f69d2ee5805
Author: Dan Hecht <dh...@cloudera.com>
AuthorDate: Thu Jun 14 13:05:49 2018 -0700

    IMPALA-6942: Reword error message to say "Failed" rather than "Cancelled"
    
    In this case, the query is failing. It happens to use the cancellation
    path to cleanup, but from a user's perspective this is a query failure
    not a cancellation. Reword the message to reflect that.
    
    Change-Id: I4d8e755aef196e5c25205094af9c8486eb899344
    Reviewed-on: http://gerrit.cloudera.org:8080/10717
    Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 be/src/service/impala-server.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/be/src/service/impala-server.cc b/be/src/service/impala-server.cc
index 771b0eb..2026842 100644
--- a/be/src/service/impala-server.cc
+++ b/be/src/service/impala-server.cc
@@ -1699,7 +1699,7 @@ void ImpalaServer::MembershipCallback(
           cancellation_entry != queries_to_cancel.end();
           ++cancellation_entry) {
         stringstream cause_msg;
-        cause_msg << "Cancelled due to unreachable impalad(s): ";
+        cause_msg << "Failed due to unreachable impalad(s): ";
         for (int i = 0; i < cancellation_entry->second.size(); ++i) {
           cause_msg << TNetworkAddressToString(cancellation_entry->second[i]);
           if (i + 1 != cancellation_entry->second.size()) cause_msg << ", ";