Riza Suminto has uploaded this change for review. ( http://gerrit.cloudera.org:8080/23168
Change subject: IMPALA-14220: CatalogServer::IsActive must not hold catalog_lock_ ...................................................................... IMPALA-14220: CatalogServer::IsActive must not hold catalog_lock_ When catalogd HA is enabled, catalogd will check whether it is the active one, via CatalogServer::IsActive, before serving each request. Calling CatalogServer::IsActive require obtaining catalog_lock_, which can contend with long catalog operation such as GatherCatalogUpdatesThread. Checking current catalog active status does not need to obtain catalog_lock_. Instead, it is sufficient to change is_active_ field from boolean to AtomicBoolean. This patch applies that change. Testing: Run and pass test_catalogd_ha.py. Change-Id: I15fb925f1eb4ea5d213075b66a676d2bc9b9e9f1 --- M be/src/catalog/catalog-server.cc M be/src/catalog/catalog-server.h M be/src/catalog/workload-management-init.cc 3 files changed, 11 insertions(+), 12 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/68/23168/1 -- To view, visit http://gerrit.cloudera.org:8080/23168 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I15fb925f1eb4ea5d213075b66a676d2bc9b9e9f1 Gerrit-Change-Number: 23168 Gerrit-PatchSet: 1 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com>
