Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11943 )
Change subject: Fix some issues in HMS and Sentry test fixtures ...................................................................... Fix some issues in HMS and Sentry test fixtures 1. The HMS or Sentry client object may be null in TearDown. If we don't check, we'll deref a null pointer if there's an error during SetUp() (e.g. if the HMS or Sentry server failed to start). 2. The HMSCatalogTest fixture should properly chain to its superclass. Change-Id: Ib0376b972fe6add6d9312aea6944c9ab1a03f25f Reviewed-on: http://gerrit.cloudera.org:8080/11943 Reviewed-by: Hao Hao <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/hms/hms_catalog-test.cc M src/kudu/sentry/sentry-test-base.h 2 files changed, 8 insertions(+), 2 deletions(-) Approvals: Hao Hao: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/11943 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib0376b972fe6add6d9312aea6944c9ab1a03f25f Gerrit-Change-Number: 11943 Gerrit-PatchSet: 2 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
