From 0e8f6b268fe4880b628d6ba4bec461d5567def29 Mon Sep 17 00:00:00 2001 From: Hiroyuki Sato Date: Thu, 9 May 2024 16:49:44 +0900 Subject: [PATCH] Use default statement instead of SuppressWarnings --- src/main/java/org/embulk/input/CommandFileInputPlugin.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) 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));