Hello Quanlong Huang, Csaba Ringhofer, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/19198

to look at the new patch set (#8).

Change subject: IMPALA-11478: Cleanup JniCatalog
......................................................................

IMPALA-11478: Cleanup JniCatalog

This change adds a new class called JniCatalogOp which extracts
the time calculation, error handling, thread annotation and logging
from the operations in JniCatalog.

Frontend test added as JniCatalogOpTest.
Manually checked thread dump for proper thread naming, OOM exception,
runtime exception and too slow and too large request warning.

Change-Id: I1932172e2d13a7aab2336661c18befb4407ec9ab
---
M fe/src/main/java/org/apache/impala/common/JniUtil.java
M fe/src/main/java/org/apache/impala/service/JniCatalog.java
A fe/src/main/java/org/apache/impala/service/JniCatalogOp.java
A fe/src/test/java/org/apache/impala/service/JniCatalogOpTest.java
4 files changed, 419 insertions(+), 286 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/98/19198/8
--
To view, visit http://gerrit.cloudera.org:8080/19198
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1932172e2d13a7aab2336661c18befb4407ec9ab
Gerrit-Change-Number: 19198
Gerrit-PatchSet: 8
Gerrit-Owner: Peter Rozsa <pro...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Peter Rozsa <pro...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>

Reply via email to