Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10488
Change subject: IMPALA-7062: fix unsafe RuntimeProfile::SortChildren() function ...................................................................... IMPALA-7062: fix unsafe RuntimeProfile::SortChildren() function The API is inherently unsafe, because std::sort() can crash if a comparator returns unstable results, e.g. because a profile counter value changes. The generality of this function is both unnecessary and dangerous. The fix is to take a snapshot of the counter values and sort based on the snapshot. Change-Id: Ie456c87c34e7d75eb7482f499e90d50de740df40 --- M be/src/runtime/coordinator-backend-state.cc M be/src/util/runtime-profile-test.cc M be/src/util/runtime-profile.cc M be/src/util/runtime-profile.h 4 files changed, 29 insertions(+), 24 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/88/10488/1 -- To view, visit http://gerrit.cloudera.org:8080/10488 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ie456c87c34e7d75eb7482f499e90d50de740df40 Gerrit-Change-Number: 10488 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>
