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

Change subject: IMPALA-13885: (addendum) Add Missing Double Quotes to Log 
Messages
......................................................................

IMPALA-13885: (addendum) Add Missing Double Quotes to Log Messages

The log messages written when the main workload management processing
loop runs are missing double quotes around field values. These
missing quotes make it difficult to identify the actual field values.

This change adds the missing double quotes around all values in order
to differentiate them from each other.

Sample log message:
I0328 14:21:00.385085 2380412 workload-management-worker.cc:806]
c748e7c938394dc9:f63162aa00000000] wrote completed queries
table="sys.impala_query_log" record_count="3" bytes="6.11 KB"
gather_time="666.758us" exec_time="269.840ms"
query_id="c748e7c938394dc9:f63162aa00000000"

Testing was completed by running the two impacted tests locally.

Change-Id: I70111fb5726fd2623aa76168d44ecfacae90fa83
Reviewed-on: http://gerrit.cloudera.org:8080/22711
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M be/src/service/workload-management-worker.cc
M tests/custom_cluster/test_query_log.py
2 files changed, 9 insertions(+), 9 deletions(-)

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

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I70111fb5726fd2623aa76168d44ecfacae90fa83
Gerrit-Change-Number: 22711
Gerrit-PatchSet: 7
Gerrit-Owner: Jason Fehr <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>

Reply via email to