Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/9535 )

Change subject: Add fragment_instance_id/query_id to important log messages and 
make them grep friendly
......................................................................

Add fragment_instance_id/query_id to important log messages and make them grep 
friendly

Debugging via going through log messages can get very tricky when
the fragment_instance_id and query_id are missing from certain important
log messages. This patch tries to add it in places where I've had issues
in the past.

This patch also tries to standardize on 'fragment_instance_id' and
'query_id' over all log messages so that it's easy to grep through the
logs while debugging.

There may be more places that I've missed, but we can fix them as we
come across them.

Change-Id: I4bb79c0cfa8cb8b99b8539e15e3c1adcb6854cf3
Reviewed-on: http://gerrit.cloudera.org:8080/9535
Reviewed-by: Sailesh Mukil <sail...@cloudera.com>
Tested-by: Impala Public Jenkins
---
M be/src/runtime/coordinator.cc
M be/src/runtime/data-stream-mgr.cc
M be/src/runtime/data-stream-sender.cc
M be/src/runtime/krpc-data-stream-mgr.cc
M be/src/runtime/krpc-data-stream-recvr.cc
M be/src/runtime/krpc-data-stream-sender.cc
6 files changed, 37 insertions(+), 30 deletions(-)

Approvals:
  Sailesh Mukil: Looks good to me, approved
  Impala Public Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/9535
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I4bb79c0cfa8cb8b99b8539e15e3c1adcb6854cf3
Gerrit-Change-Number: 9535
Gerrit-PatchSet: 3
Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Lars Volker <l...@cloudera.com>
Gerrit-Reviewer: Michael Ho <k...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com>

Reply via email to