Hello Michael Ho, Lars Volker, Dan Hecht,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/9535

to look at the new patch set (#2).

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
---
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(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/35/9535/2
--
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: newpatchset
Gerrit-Change-Id: I4bb79c0cfa8cb8b99b8539e15e3c1adcb6854cf3
Gerrit-Change-Number: 9535
Gerrit-PatchSet: 2
Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com>
Gerrit-Reviewer: Lars Volker <l...@cloudera.com>
Gerrit-Reviewer: Michael Ho <k...@cloudera.com>

Reply via email to