Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3944: Fix crash in SimpleScheduler ......................................................................
IMPALA-3944: Fix crash in SimpleScheduler SimpleScheduler::BackendConfig maintains two maps, hostname->IP and IP->[list of backends]. In situations where multiple backends run on a single host, BackendConfig::RemoveBackend was erroneously removing the hostname->IP mapping, even when the list of backends for a host was not empty. The fix is to only remove this mapping when the last backend of a host gets removed. I will push a test in a separate change. Change-Id: I1c2c58c2f9e7de0936aba03dcce86cdc4b31a866 Reviewed-on: http://gerrit.cloudera.org:8080/4102 Reviewed-by: Lars Volker <[email protected]> Reviewed-by: Matthew Jacobs <[email protected]> Tested-by: Internal Jenkins --- M be/src/scheduling/simple-scheduler.cc 1 file changed, 15 insertions(+), 3 deletions(-) Approvals: Lars Volker: Looks good to me, but someone else must approve Matthew Jacobs: Looks good to me, approved Internal Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4102 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1c2c58c2f9e7de0936aba03dcce86cdc4b31a866 Gerrit-PatchSet: 5 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Lars Volker <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Marcel Kornacker <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]>
