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 (#6). 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, 404 insertions(+), 283 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/98/19198/6 -- 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: 6 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>