Henry Robinson has uploaded a new patch set (#2). 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 --- 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 D be/src/scheduling/scheduler.h M be/src/scheduling/simple-scheduler-test-util.cc M be/src/scheduling/simple-scheduler-test-util.h M be/src/scheduling/simple-scheduler-test.cc M be/src/scheduling/simple-scheduler.cc M 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, 238 insertions(+), 301 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/49/6149/2 -- To view, visit http://gerrit.cloudera.org:8080/6149 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I70e0b002fa56d3ba1c3b34f03ae05f4042ac309e Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Henry Robinson <he...@cloudera.com> Gerrit-Reviewer: Henry Robinson <he...@cloudera.com> Gerrit-Reviewer: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Marcel Kornacker <mar...@cloudera.com>