Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14710
Change subject: IMPALA-9125: generalize finding DataSink from other fragment ...................................................................... IMPALA-9125: generalize finding DataSink from other fragment This is a minor refactor that makes the existing mechanism for finding PlanRootSink more general: * Status-returning API to reduce boilerplate in caller. * Avoid storing PlanRootSink as a member - this doesn't scale nicely to multiple subclasses. Instead use an accessor method. Change-Id: I3c91cadf9ecfde9cc3235bd71412ba10ee37bc4e --- M be/src/runtime/coordinator.cc M be/src/runtime/fragment-instance-state.cc M be/src/runtime/fragment-instance-state.h M be/src/runtime/query-state.cc M be/src/runtime/query-state.h 5 files changed, 26 insertions(+), 22 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/10/14710/1 -- To view, visit http://gerrit.cloudera.org:8080/14710 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I3c91cadf9ecfde9cc3235bd71412ba10ee37bc4e Gerrit-Change-Number: 14710 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>