[IMPALA-6993] Don't print status stack trace when propagating thrift status in Coordinator::BackendState::Exec()

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Impala 3.0
• Fix Version/s: Impala 2.13.0, Impala 3.1.0
• Component/s: Backend
• Target Version: Product Backlog
• Epic Color: ghx-label-5
• Assignee: Daniel Hecht
• Reporter: Daniel Hecht
• Created: 08/May/18 19:12
• Updated: 09/May/18 16:33
• Resolved: 09/May/18 16:33

Description

We should use Status::Expected() here, just like it's used above. The stack trace isn't interesting and the error is expected once we get down this path. (I'm not sure why we don't just use exec_status though, but presumably the prefix was added for a reason).

Status exec_status = Status(thrift_result.status);
if (!exec_status.ok()) {
  const string& err_msg = Substitute(ERR_TEMPLATE, PrintId(query_id_),
  exec_status.msg().GetFullMessageDetails());
  VLOG_QUERY << err_msg;
  status_ = Status(err_msg);
  return;
}