Merge pull request #97 from treasure-data/fix_actTypeIds_required

Hotfix issue actTypeIds is required
treasure-data · Dec 3, 2019 · e5a5f08 · e5a5f08
2 parents 40560d7 + 806bf21
commit e5a5f08
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/main/java/org/embulk/input/marketo/delegate/ActivityBulkExtractInputPlugin.java b/src/main/java/org/embulk/input/marketo/delegate/ActivityBulkExtractInputPlugin.java
@@ -39,6 +39,7 @@ public interface PluginTask extends MarketoBaseBulkExtractInputPlugin.PluginTask
         @ConfigDefault("[]")
         List<String> getActivityTypeIds();
 
+        @Config("act_type_ids")
         @ConfigDefault("[]")
         List<Integer> getActTypeIds();
 
@@ -63,11 +64,9 @@ public void validateInputTask(PluginTask task)
                 // ignorable if unable to get activity type ids. If thing gone wrong, the bulk extract will throw errors
             }
 
+            // task will use getActTypeIds instead of getActivityTypeIds method
             task.setActTypeIds(activityIds);
         }
-        else {
-            task.setActTypeIds(new ArrayList<Integer>());
-        }
         super.validateInputTask(task);
     }