Sailesh Mukil has posted comments on this change. Change subject: PREVIEW: IMPALA-2550 Introduce query-wide execution context. ......................................................................
Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/3817/3/be/src/runtime/query-exec-state.h File be/src/runtime/query-exec-state.h: PS3, Line 87: num_active_fragment_instances_ This is not really used right now. May remove it if there doesn't seem to be any use for it. http://gerrit.cloudera.org:8080/#/c/3817/3/be/src/service/fragment-mgr.h File be/src/service/fragment-mgr.h: PS3, Line 81: class ScopedQueryExecStateRef { I don't know if this is a good way, or if we should have every caller call GetQES() and ReturnQES() explicitly. Open for suggestions. -- To view, visit http://gerrit.cloudera.org:8080/3817 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I892091d6401acb2ea91ccb1623af54c6f9635e6c Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Sailesh Mukil <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]> Gerrit-HasComments: Yes
