Fredy Wijaya has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/12241 )
Change subject: IMPALA-8073: Fix FE tests that leak HMS connections ...................................................................... IMPALA-8073: Fix FE tests that leak HMS connections This patch fixes FE tests that leak the HMS connections. I was to reproduce the issue by in SentryTestProxy by looping the test multiple times. Testing: - Ran SentryProxyTest in a loop without any issue. - Ran all FE tests Change-Id: I56930bc5f7a7bda4db11d4447461f907b1017b91 --- M fe/src/main/java/org/apache/impala/catalog/Catalog.java M fe/src/test/java/org/apache/impala/analysis/AuditingTest.java M fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java M fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java M fe/src/test/java/org/apache/impala/catalog/CatalogObjectToFromThriftTest.java M fe/src/test/java/org/apache/impala/catalog/CatalogTest.java M fe/src/test/java/org/apache/impala/catalog/CatalogdTableInvalidatorTest.java M fe/src/test/java/org/apache/impala/catalog/PartialCatalogInfoTest.java M fe/src/test/java/org/apache/impala/common/FrontendTestBase.java M fe/src/test/java/org/apache/impala/testutil/BlockIdGenerator.java M fe/src/test/java/org/apache/impala/testutil/ImpaladTestCatalog.java M fe/src/test/java/org/apache/impala/util/SentryProxyTest.java 12 files changed, 205 insertions(+), 180 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/41/12241/5 -- To view, visit http://gerrit.cloudera.org:8080/12241 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I56930bc5f7a7bda4db11d4447461f907b1017b91 Gerrit-Change-Number: 12241 Gerrit-PatchSet: 5 Gerrit-Owner: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
