Riza Suminto has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17819
Change subject: IMPALA-10883: Do not override existing counters with empty profile ...................................................................... IMPALA-10883: Do not override existing counters with empty profile Some profile information was missing when gen_experimental_profile flag is enabled. This is because profile aggregation in the coordinator does not anticipate a partial update from a backend. From a backend perspective, if a fragment instance has sent its final report, that instance will not participate in the subsequent profile report. Therefore, the aggregated counters that belong to the finished instance will be empty. This patch adds empty checks in the aggregation of input_profile_names_, TAggTimeSeriesCounter, and TAggEventSequence to prevent the existing value from being overridden by an empty profile. Testing: - Add BE test CountersTest.PartialUpdate Change-Id: I9bb179bf739ffaa4e5ec8dc911480ac835ae387f --- M be/src/util/runtime-profile-test.cc M be/src/util/runtime-profile.cc 2 files changed, 144 insertions(+), 5 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/19/17819/1 -- To view, visit http://gerrit.cloudera.org:8080/17819 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I9bb179bf739ffaa4e5ec8dc911480ac835ae387f Gerrit-Change-Number: 17819 Gerrit-PatchSet: 1 Gerrit-Owner: Riza Suminto <[email protected]>
