Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 15289200CA3 for ; Thu, 1 Jun 2017 23:52:25 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 139DB160BC4; Thu, 1 Jun 2017 21:52:25 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 5945C160BC1 for ; Thu, 1 Jun 2017 23:52:24 +0200 (CEST) Received: (qmail 21344 invoked by uid 500); 1 Jun 2017 21:52:23 -0000 Mailing-List: contact reviews-help@impala.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list reviews@impala.incubator.apache.org Received: (qmail 21333 invoked by uid 99); 1 Jun 2017 21:52:23 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd1-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 01 Jun 2017 21:52:23 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd1-us-west.apache.org (ASF Mail Server at spamd1-us-west.apache.org) with ESMTP id 00410CA9F0 for ; Thu, 1 Jun 2017 21:52:23 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 0.363 X-Spam-Level: X-Spam-Status: No, score=0.363 tagged_above=-999 required=6.31 tests=[RDNS_DYNAMIC=0.363, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new, port 10024) with ESMTP id R_fesBAw6iLH for ; Thu, 1 Jun 2017 21:52:22 +0000 (UTC) Received: from ip-10-146-233-104.ec2.internal (ec2-75-101-130-251.compute-1.amazonaws.com [75.101.130.251]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTPS id 0B7745F5A2 for ; Thu, 1 Jun 2017 21:52:21 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by ip-10-146-233-104.ec2.internal (8.14.4/8.14.4) with ESMTP id v51LqKEr009432; Thu, 1 Jun 2017 21:52:20 GMT Message-Id: <201706012152.v51LqKEr009432@ip-10-146-233-104.ec2.internal> Date: Thu, 1 Jun 2017 21:52:20 +0000 From: "Impala Public Jenkins (Code Review)" To: Bharath Vissapragada , impala-cr@cloudera.com, reviews@impala.incubator.apache.org X-Gerrit-MessageType: merged Subject: =?UTF-8?Q?=5BImpala-ASF-CR=5D_IMPALA-5411=3A_Avoid_log_spew_from_GetRuntimeProfileStr=0A?= X-Gerrit-Change-Id: I2d0ae8c161346b4ec8c865bc23c5c1cd49c1ce7b X-Gerrit-ChangeURL: X-Gerrit-Commit: bbc3ce1b985d56b9fb2ab95656b7d0ae2b6126ca In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Content-Disposition: inline User-Agent: Gerrit/2.12.7 archived-at: Thu, 01 Jun 2017 21:52:25 -0000 Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5411: Avoid log spew from GetRuntimeProfileStr ...................................................................... IMPALA-5411: Avoid log spew from GetRuntimeProfileStr GetRuntimeProfileStr can potentially log lots of spam to the coordinator logs if there are clients continuously polling the web endpoints when the table metadata is still loading. The fix is to use an appopriate Status constructor that doesn't log it. Change-Id: I2d0ae8c161346b4ec8c865bc23c5c1cd49c1ce7b Reviewed-on: http://gerrit.cloudera.org:8080/7042 Reviewed-by: Henry Robinson Tested-by: Impala Public Jenkins --- M be/src/service/impala-server.cc 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Impala Public Jenkins: Verified Henry Robinson: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/7042 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2d0ae8c161346b4ec8c865bc23c5c1cd49c1ce7b Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Bharath Vissapragada Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Henry Robinson Gerrit-Reviewer: Impala Public Jenkins