Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/4188
Change subject: Move ExecOption profile helpers to RuntimeProfile ...................................................................... Move ExecOption profile helpers to RuntimeProfile This is groundwork for IMPALA-3567, which will move some logic that previously resided in ExecNodes into DataSinks. We want to report ExecOption strings consistently in both ExecNodes and DataSinks, so that logic needs to move to a shared place (e.g. the RuntimeProfile itself). I ran the patch through clang-format, which changed line wrapping and indentation of a few surrounding lines. Change-Id: I21c1dda8f8a1d92172bf59fbc1070a6834e61913 --- M be/src/exec/aggregation-node.cc M be/src/exec/blocking-join-node.cc M be/src/exec/exec-node.cc M be/src/exec/exec-node.h M be/src/exec/hash-join-node.cc M be/src/exec/hdfs-scan-node.cc M be/src/exec/partitioned-aggregation-node.cc M be/src/exec/partitioned-hash-join-node.cc M be/src/exec/sort-node.cc M be/src/exec/topn-node.cc M be/src/util/runtime-profile.cc M be/src/util/runtime-profile.h 12 files changed, 112 insertions(+), 98 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/88/4188/1 -- To view, visit http://gerrit.cloudera.org:8080/4188 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I21c1dda8f8a1d92172bf59fbc1070a6834e61913 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim Armstrong <[email protected]>
