diff --git a/src/main/java/org/embulk/input/CommandFileInputPlugin.java b/src/main/java/org/embulk/input/CommandFileInputPlugin.java index 3f7720a..792b5fa 100644 --- a/src/main/java/org/embulk/input/CommandFileInputPlugin.java +++ b/src/main/java/org/embulk/input/CommandFileInputPlugin.java @@ -71,7 +71,6 @@ public void cleanup(TaskSource taskSource, List successTaskReports) { } - @SuppressWarnings("MissingSwitchDefault") @Override public TransactionalFileInput open(TaskSource taskSource, int taskIndex) { final TaskMapper taskMapper = CONFIG_MAPPER_FACTORY.createTaskMapper(); @@ -91,6 +90,9 @@ public TransactionalFileInput open(TaskSource taskSource, int taskIndex) { case "stderr": builder.redirectOutput(ProcessBuilder.Redirect.INHERIT); break; + default: + throw new IllegalStateException(String.format( + "Unknown 'pipe' option '%s'. It must be either 'stdout' or 'stderr'", task.getPipe())); } try { @@ -105,6 +107,9 @@ public TransactionalFileInput open(TaskSource taskSource, int taskIndex) { case "stderr": stream = process.getErrorStream(); break; + default: + throw new IllegalStateException(String.format( + "Unknown 'pipe' option '%s'. It must be either 'stdout' or 'stderr'", task.getPipe())); } PluginFileInput input = new PluginFileInput(task, new ProcessWaitInputStream(stream, process));