impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lars Volker (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-4494: Fix crash in SimpleScheduler
Date Fri, 18 Nov 2016 20:09:01 GMT
Lars Volker has uploaded a new patch set (#5).

Change subject: IMPALA-4494: Fix crash in SimpleScheduler

IMPALA-4494: Fix crash in SimpleScheduler

The scheduler maintains a local list of active backends, which is
updated through messages from the statestore. Even the local backend
enters this list by registering with the statestore and being included
in a statestore update message. Thus, during restarts it can happen that
a query gets scheduled with exec_at_coord set to true, while the local
backend has not been registered with the scheduler. In this case the IP
address lookup in the internal BackendConfig fails and an empty IP
address is returned, leading to a nullptr dereference down the line.

This change adds an additional check when handling updates from the
statestore to make sure that the backend config is either empty or
contains the local backend as well. It also changes scheduling when
exec_at_coord is true to always use the local backend, irrespective of
whether it is present in the backend config.

Change-Id: I6e1196a2fa47e5954c4a190aa326c135d039a77f
M be/src/scheduling/
M be/src/scheduling/
2 files changed, 85 insertions(+), 24 deletions(-)

  git pull ssh:// refs/changes/27/5127/5
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6e1196a2fa47e5954c4a190aa326c135d039a77f
Gerrit-PatchSet: 5
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Lars Volker <>
Gerrit-Reviewer: Henry Robinson <>
Gerrit-Reviewer: Lars Volker <>
Gerrit-Reviewer: Tim Armstrong <>

View raw message