Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16122 )
Change subject: IMPALA-9692 (part 4): Rename QuerySchedule to ScheduleState ...................................................................... IMPALA-9692 (part 4): Rename QuerySchedule to ScheduleState This is the final patch in the refactor of QuerySchedule for the single admission controller work. It was kept separate for ease of reviewing. This patch renames QuerySchedule and its related classes (FInstanceExecParams, BackendExecParams, and FragmentExecParams) to use the name 'ScheduleState', reflecting the fact that they are used as containers for intermediate info about a query during scheduling. The messages that are included in the QuerySchedulePB struct retain the 'ExecParams' name, reflecting the fact that they are used by the coordinator to start execution. Change-Id: I57069c4a426f3e697df7e2a07754d063bdea26f7 Reviewed-on: http://gerrit.cloudera.org:8080/16122 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M be/src/scheduling/CMakeLists.txt M be/src/scheduling/admission-controller-test.cc M be/src/scheduling/admission-controller.cc M be/src/scheduling/admission-controller.h R be/src/scheduling/schedule-state.cc R be/src/scheduling/schedule-state.h M be/src/scheduling/scheduler-test-util.h M be/src/scheduling/scheduler.cc M be/src/scheduling/scheduler.h 9 files changed, 631 insertions(+), 625 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/16122 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I57069c4a426f3e697df7e2a07754d063bdea26f7 Gerrit-Change-Number: 16122 Gerrit-PatchSet: 6 Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Sahil Takiar <stak...@cloudera.com> Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com>
