Quanlong Huang has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/23535


Change subject: IMPALA-14494: Tag catalogd logs of GetPartialCatalogObject 
requests with correct query ids
......................................................................

IMPALA-14494: Tag catalogd logs of GetPartialCatalogObject requests with 
correct query ids

Catalogd logs of GetPartialCatalogObject requests are not tagged with
correct query ids. Instead, the query id that is previously using that
thread is printed in the logs.

This is fixed by sending the query id in GetPartialCatalogObject
requests so catalogd can tag the responding thread with it.

Some codes are copied from Michael Smith's patch: 
https://gerrit.cloudera.org/c/22738/

Tested by enabling TRACE logging in org.apache.impala.common.JniUtil to
verify logs of GetPartialCatalogObject requests.

I20251014 09:39:39.685225 342587 JniUtil.java:165] 
964e37e9303d6f8a:eab7096000000000] getPartialCatalogObject request: Getting 
partial catalog object of CATALOG_SERVICE_ID
I20251014 09:39:39.690346 342587 JniUtil.java:176] 
964e37e9303d6f8a:eab7096000000000] Finished getPartialCatalogObject request: 
Getting partial catalog object of CATALOG_SERVICE_ID. Time spent: 5ms
I20251014 09:39:39.699471 342587 JniUtil.java:165] 
964e37e9303d6f8a:eab7096000000000] getPartialCatalogObject request: Getting 
partial catalog object of DATABASE:functional
I20251014 09:39:39.701821 342587 JniUtil.java:176] 
964e37e9303d6f8a:eab7096000000000] Finished getPartialCatalogObject request: 
Getting partial catalog object of DATABASE:functional. Time spent: 2ms
I20251014 09:39:39.711462 341074 TAcceptQueueServer.cpp:368] New connection to 
server CatalogService from client <Host: 127.0.0.1 Port: 42084>
I20251014 09:39:39.719146 342588 JniUtil.java:165] 
964e37e9303d6f8a:eab7096000000000] getPartialCatalogObject request: Getting 
partial catalog object of TABLE:functional.alltypestiny

Change-Id: Ie63363ac60e153e3a69f2a4cf6a0f4ce10701674
---
M be/src/catalog/catalog.cc
M be/src/service/fe-support.cc
M common/thrift/CatalogService.thrift
3 files changed, 18 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/35/23535/1
--
To view, visit http://gerrit.cloudera.org:8080/23535
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie63363ac60e153e3a69f2a4cf6a0f4ce10701674
Gerrit-Change-Number: 23535
Gerrit-PatchSet: 1
Gerrit-Owner: Quanlong Huang <[email protected]>

Reply via email to