This is an automated email from the ASF dual-hosted git repository. wzhou pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit eea3594e96e3d3fc0cf9d07be651d686537aaf36 Author: Riza Suminto <[email protected]> AuthorDate: Fri Sep 20 19:31:28 2024 -0700 IMPALA-13397: Use TUniqueIdUtil.PrintId in HeartbeatContext.toString Impala begin to consistently use PrintId to print TUniqueId since IMPALA-13252. This patch add one more PrintId in HeartbeatContext.toString to pass TestBannedLogMessage.test_no_tuniqueid. Change-Id: Ic9395fab07b5f85342da692a350ca21ec502a1ea Reviewed-on: http://gerrit.cloudera.org:8080/21840 Reviewed-by: Michael Smith <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- fe/src/main/java/org/apache/impala/common/TransactionKeepalive.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fe/src/main/java/org/apache/impala/common/TransactionKeepalive.java b/fe/src/main/java/org/apache/impala/common/TransactionKeepalive.java index 126937826..24572cc20 100644 --- a/fe/src/main/java/org/apache/impala/common/TransactionKeepalive.java +++ b/fe/src/main/java/org/apache/impala/common/TransactionKeepalive.java @@ -34,6 +34,7 @@ import org.apache.impala.catalog.MetaStoreClientPool.MetaStoreClient; import org.apache.impala.common.TransactionException; import org.apache.impala.compat.MetastoreShim; import org.apache.impala.thrift.TQueryCtx; +import org.apache.impala.util.TUniqueIdUtil; import org.apache.log4j.Logger; import com.google.common.base.Preconditions; @@ -79,7 +80,7 @@ public class TransactionKeepalive { } public String toString() { - if (queryCtx != null) return queryCtx.query_id.toString(); + if (queryCtx != null) return TUniqueIdUtil.PrintId(queryCtx.query_id); return cause; } }
