Michael Ho has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12651
Change subject: IMPALA-8274: Fix iteration of profiles in ApplyExecStatusReport() ...................................................................... IMPALA-8274: Fix iteration of profiles in ApplyExecStatusReport() The coordinator skips over any stale or duplicated status reports of fragment instances. In the previous implementation, the index pointing into the vector of Thrift profiles wasn't updated when skipping over a status report. This breaks the assumption that the status reports and thrift profiles vectors have one-to-one correspondence. Consequently, we may end up passing the wrong profile to InstanceStats::Update(), leading to random crashes. This change fixes the problem above by using iterators to iterate through the status reports and thrift profiles vectors and ensures that both iterators are updated on every iteration of the loop. Change-Id: I8bce426c7d08ffbf0f8cd26889262243a52cc752 --- M be/src/runtime/coordinator-backend-state.cc M tests/custom_cluster/test_rpc_timeout.py 2 files changed, 32 insertions(+), 17 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/51/12651/1 -- To view, visit http://gerrit.cloudera.org:8080/12651 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I8bce426c7d08ffbf0f8cd26889262243a52cc752 Gerrit-Change-Number: 12651 Gerrit-PatchSet: 1 Gerrit-Owner: Michael Ho <[email protected]>
