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>

Reply via email to