This is an automated email from the ASF dual-hosted git repository. alexey pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/kudu.git
commit 46d2cdf2b52a9511783d28b4fbecda847047070b Author: Alexey Serbin <[email protected]> AuthorDate: Thu May 28 21:58:00 2020 -0700 [scanner] output the essential error message first While digging through logs as they seen in kudu-tserver Web UI, I found it hard to get to the essence of the warning message on setting up a new scanner: it contained a very long string prefix with the information about the scan projection. This patch addresses that, moving the essential information on the status into the beginning of the message. Otherwise, this patch doesn't contain any functional modifications. Change-Id: I06e4ed516dcd2b520f26ddc580aed00be239df04 Reviewed-on: http://gerrit.cloudera.org:8080/15999 Tested-by: Kudu Jenkins Reviewed-by: Bankim Bhavsar <[email protected]> --- src/kudu/tserver/tablet_service.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/kudu/tserver/tablet_service.cc b/src/kudu/tserver/tablet_service.cc index 91e3477..4a5c040 100644 --- a/src/kudu/tserver/tablet_service.cc +++ b/src/kudu/tserver/tablet_service.cc @@ -2647,8 +2647,8 @@ Status TabletServiceImpl::HandleNewScanRequest(TabletReplica* replica, TRACE("Iterator init: $0", s.ToString()); if (PREDICT_FALSE(!s.ok())) { - LOG(WARNING) << Substitute("Error setting up scanner with request $0: $1", - SecureShortDebugString(*req), s.ToString()); + LOG(WARNING) << Substitute("Error setting up scanner with request: $0: $1", + s.ToString(), SecureShortDebugString(*req)); // If the replica has been stopped, e.g. due to disk failure, return // TABLET_FAILED so the scan can be handled appropriately (fail over to // another tablet server if fault-tolerant).
