Impala Public Jenkins has submitted this change and it was merged. Change subject: Remove Scheduler abstract interface, rename SimpleScheduler -> Scheduler ......................................................................
Remove Scheduler abstract interface, rename SimpleScheduler -> Scheduler For the last five years we've had the Scheduler superclass with exactly one implementation, SimpleScheduler. This patch collapses that hierarchy into just one concrete implementation called Scheduler. Also fixed up some includes in (the new) scheduler.h based on the include-what-you-use tool. Change-Id: I70e0b002fa56d3ba1c3b34f03ae05f4042ac309e Reviewed-on: http://gerrit.cloudera.org:8080/6149 Reviewed-by: Henry Robinson <[email protected]> Tested-by: Impala Public Jenkins --- M be/src/runtime/exec-env.cc M be/src/scheduling/CMakeLists.txt M be/src/scheduling/admission-controller.cc M be/src/scheduling/query-schedule.h R be/src/scheduling/scheduler-test-util.cc R be/src/scheduling/scheduler-test-util.h R be/src/scheduling/scheduler-test.cc R be/src/scheduling/scheduler.cc M be/src/scheduling/scheduler.h D be/src/scheduling/simple-scheduler.h M be/src/service/impala-server.cc M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java M fe/src/main/java/org/apache/impala/util/MembershipSnapshot.java 13 files changed, 633 insertions(+), 695 deletions(-) Approvals: Impala Public Jenkins: Verified Henry Robinson: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/6149 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I70e0b002fa56d3ba1c3b34f03ae05f4042ac309e Gerrit-PatchSet: 4 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Henry Robinson <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Marcel Kornacker <[email protected]>
