Github user m-hogue commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2131#discussion_r137437479
  
    --- Diff: 
nifi-nar-bundles/nifi-grpc-bundle/nifi-grpc-processors/src/main/java/org/apache/nifi/processors/grpc/ListenGRPC.java
 ---
    @@ -82,25 +83,29 @@
         // properties
         public static final PropertyDescriptor PROP_SERVICE_PORT = new 
PropertyDescriptor.Builder()
                 .name("Local gRPC service port")
    +            .displayName("Local gRPC Service Port")
                 .description("The local port that the gRPC service will listen 
on.")
                 .required(true)
                 .addValidator(StandardValidators.PORT_VALIDATOR)
                 .build();
         public static final PropertyDescriptor PROP_USE_SECURE = new 
PropertyDescriptor.Builder()
    -            .name("Use SSL/TLS")
    -            .description("Whether or not to use SSL/TLS to send the 
contents of the gRPC messages.")
    +            .name("Use TLS")
    --- End diff --
    
    My rationale on that name change was that `ListenGRPC` hasn't appeared in a 
formal nifi release yet (I added it in the current baseline), so i went with 
the name change. However, i'm good with whichever way you decide.


---

Reply via email to