Merge pull request #2311 from metamx/commentsIn2309

Minor field name change in RemoteTaskRunnerFactory to be more descriptive
This commit is contained in:
Fangjin Yang 2016-01-20 19:27:49 -08:00
commit 500c11a35a
1 changed files with 3 additions and 3 deletions

View File

@ -53,7 +53,7 @@ public class RemoteTaskRunnerFactory implements TaskRunnerFactory<RemoteTaskRunn
private final ScheduledExecutorService cleanupExec;
private final SimpleResourceManagementConfig config;
private final ResourceManagementSchedulerConfig resourceManagementSchedulerConfig;
private final ScheduledExecutorService exec;
private final ScheduledExecutorService resourceManagementExec;
@Inject
public RemoteTaskRunnerFactory(
@ -77,7 +77,7 @@ public class RemoteTaskRunnerFactory implements TaskRunnerFactory<RemoteTaskRunn
this.cleanupExec = factory.create(1, "RemoteTaskRunner-Scheduled-Cleanup--%d");
this.config = config;
this.resourceManagementSchedulerConfig = resourceManagementSchedulerConfig;
this.exec = factory.create(1, "RemoteTaskRunner-ResourceManagement--%d");
this.resourceManagementExec = factory.create(1, "RemoteTaskRunner-ResourceManagement--%d");
}
@Override
@ -89,7 +89,7 @@ public class RemoteTaskRunnerFactory implements TaskRunnerFactory<RemoteTaskRunn
config,
workerConfigRef,
resourceManagementSchedulerConfig,
exec
resourceManagementExec
);
} else {
resourceManagementStrategy = new NoopResourceManagementStrategy<>();