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

Change subject: IMPALA-11948: Remove operation logging from 
getCatalogServerMetrics
......................................................................

IMPALA-11948: Remove operation logging from getCatalogServerMetrics

IMPALA-11478 added operation-level logging to every catalog method in
JniCatalog, but getCatalogServerMetrics operation is called many times
by catalogd itself. Start and finish logs for this operation are useless,
therefore only the warnings about size and elapsed time should remain.

Tests:
   - manually validated that warning-level logs are still generated
   - manually validated that start/finish log entries are not generated

Change-Id: Ifd01e9f42ccc6a4adce41caf8884ea3c2c781452
Reviewed-on: http://gerrit.cloudera.org:8080/19557
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M fe/src/main/java/org/apache/impala/common/JniUtil.java
M fe/src/main/java/org/apache/impala/service/JniCatalog.java
M fe/src/main/java/org/apache/impala/service/JniCatalogOp.java
3 files changed, 69 insertions(+), 22 deletions(-)

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

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifd01e9f42ccc6a4adce41caf8884ea3c2c781452
Gerrit-Change-Number: 19557
Gerrit-PatchSet: 7
Gerrit-Owner: Peter Rozsa <[email protected]>
Gerrit-Reviewer: Daniel Becker <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>

Reply via email to