Github user alopresto commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2131#discussion_r137413455
--- 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 --
The `name` value needs to stay the same so that templates and flows which
referenced the previous version will continue to work. If this value changes,
existing templates and flows will not be read correctly and these values will
be lost. This is why `displayName` was introduced.
---
