Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19198 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/19198 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 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(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I1932172e2d13a7aab2336661c18befb4407ec9ab Gerrit-Change-Number: 19198 Gerrit-PatchSet: 10 Gerrit-Owner: Peter Rozsa <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]>
