From 25e330780ca35fa44046888b99d076b2a3a46e02 Mon Sep 17 00:00:00 2001 From: Gian Merlino Date: Tue, 20 Aug 2013 13:38:52 -0700 Subject: [PATCH] Simplify AbstractTask constructor --- .../com/metamx/druid/indexing/common/task/AbstractTask.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/indexing-service/src/main/java/com/metamx/druid/indexing/common/task/AbstractTask.java b/indexing-service/src/main/java/com/metamx/druid/indexing/common/task/AbstractTask.java index e4bab80958..02d48613b5 100644 --- a/indexing-service/src/main/java/com/metamx/druid/indexing/common/task/AbstractTask.java +++ b/indexing-service/src/main/java/com/metamx/druid/indexing/common/task/AbstractTask.java @@ -58,11 +58,7 @@ public abstract class AbstractTask implements Task protected AbstractTask(String id, String groupId, String dataSource, Interval interval) { - this.id = Preconditions.checkNotNull(id, "id"); - this.groupId = Preconditions.checkNotNull(groupId, "groupId"); - this.taskResource = new TaskResource(id, 1); - this.dataSource = Preconditions.checkNotNull(dataSource, "dataSource"); - this.interval = Optional.fromNullable(interval); + this(id, groupId, new TaskResource(id, 1), dataSource, interval); } protected AbstractTask(String id, String groupId, TaskResource taskResource, String dataSource, Interval interval) -- GitLab