NIFI-12011: Incorporated review comments

mattyb149 · Sep 12, 2023 · 3461cef · 3461cef
1 parent 0aa7c83
commit 3461cef
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/...standard-processors/src/main/java/org/apache/nifi/processors/standard/ExecuteProcess.java b/...standard-processors/src/main/java/org/apache/nifi/processors/standard/ExecuteProcess.java
@@ -85,7 +85,7 @@
 @WritesAttributes({
     @WritesAttribute(attribute = "command", description = "Executed command"),
     @WritesAttribute(attribute = "command.arguments", description = "Arguments of the command"),
-    @WritesAttribute(attribute = "mime.type", description = "Sets the MIME type of the output if the 'Mime Type' property is set and 'Batch Duration' is not set")
+    @WritesAttribute(attribute = "mime.type", description = "Sets the MIME type of the output if the 'Output MIME Type' property is set and 'Batch Duration' is not set")
 })
 public class ExecuteProcess extends AbstractProcessor {
 
@@ -149,8 +149,8 @@ public class ExecuteProcess extends AbstractProcessor {
       .build();
 
     static final PropertyDescriptor MIME_TYPE = new PropertyDescriptor.Builder()
-            .name("mime-type")
-            .displayName("Mime Type")
+            .name("Output MIME type")
+            .displayName("Output MIME Type")
             .description("Specifies the value to set for the \"mime.type\" attribute. This property is ignored if 'Batch Duration' is set.")
             .required(false)
             .addValidator(StandardValidators.NON_EMPTY_VALIDATOR)
@@ -303,7 +303,7 @@ public void process(final OutputStream flowFileOut) throws IOException {
             getLogger().error("Failed to read data from Process, so will not generate FlowFile");
         } else {
             // add command, arguments, and MIME type as attributes
-            HashMap<String,String> attributes = new HashMap<>();
+            Map<String,String> attributes = new HashMap<>();
             attributes.put(ATTRIBUTE_COMMAND, command);
             if(arguments != null) {
                 attributes.put(ATTRIBUTE_COMMAND_ARGS, arguments);

diff --git a/...rd-processors/src/main/java/org/apache/nifi/processors/standard/ExecuteStreamCommand.java b/...rd-processors/src/main/java/org/apache/nifi/processors/standard/ExecuteStreamCommand.java
@@ -161,7 +161,7 @@
         @WritesAttribute(attribute = "execution.command.args", description = "The semi-colon delimited list of arguments. Sensitive properties will be masked"),
         @WritesAttribute(attribute = "execution.status", description = "The exit status code returned from executing the command"),
         @WritesAttribute(attribute = "execution.error", description = "Any error messages returned from executing the command"),
-        @WritesAttribute(attribute = "mime.type", description = "Sets the MIME type of the output if the 'Mime Type' property is set and 'Output Destination Attribute' is not set")})
+        @WritesAttribute(attribute = "mime.type", description = "Sets the MIME type of the output if the 'Output MIME Type' property is set and 'Output Destination Attribute' is not set")})
 @Restricted(
         restrictions = {
                 @Restriction(
@@ -277,8 +277,8 @@ public class ExecuteStreamCommand extends AbstractProcessor {
             .build();
 
     static final PropertyDescriptor MIME_TYPE = new PropertyDescriptor.Builder()
-            .name("mime-type")
-            .displayName("Mime Type")
+            .name("Output MIME Type")
+            .displayName("Output MIME Type")
             .description("Specifies the value to set for the \"mime.type\" attribute. This property is ignored if 'Output Destination Attribute' is set.")
             .required(false)
             .addValidator(StandardValidators.NON_EMPTY_VALIDATOR)