Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22620 )
Change subject: IMPALA-13860: Fix DCHECK hit in cluster-membership-mgr.cc ...................................................................... IMPALA-13860: Fix DCHECK hit in cluster-membership-mgr.cc Enabling graceful shutdown in test_coord_only_pool_exec_groups reveals a DCHECK hit caused by two back-two-back call to RemoveExecutorAndGroup during graceful shutdown. This patch fix it by turning the DHCECK into if and VLOG(1). Testing: - Pass test_coord_only_pool_exec_groups with graceful shutdown after the change without any FATAL log. Change-Id: If678ae472bade50c18842df9e98c536fb9f1fe9c Reviewed-on: http://gerrit.cloudera.org:8080/22620 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/scheduling/cluster-membership-mgr.cc M tests/custom_cluster/test_admission_controller.py 2 files changed, 12 insertions(+), 3 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/22620 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If678ae472bade50c18842df9e98c536fb9f1fe9c Gerrit-Change-Number: 22620 Gerrit-PatchSet: 6 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]>
