greyp9 commented on code in PR #6131:
URL: https://github.com/apache/nifi/pull/6131#discussion_r921359440
##########
nifi-nar-bundles/nifi-kafka-bundle/nifi-kafka-2-6-processors/src/main/java/org/apache/nifi/processors/kafka/pubsub/PublishKafkaRecord_2_6.java:
##########
@@ -271,7 +273,21 @@ public class PublishKafkaRecord_2_6 extends
AbstractProcessor implements Verifia
.defaultValue("UTF-8")
.required(false)
.build();
-
+ static final PropertyDescriptor PUBLISH_STRATEGY = new
PropertyDescriptor.Builder()
+ .name("publish-strategy")
+ .displayName("Publish Strategy")
+ .description("The format used to publish the outgoing FlowFile
record to Kafka.")
+ .required(true)
+ .defaultValue(WRITE_VALUE_ONLY.getValue())
+ .allowableValues(WRITE_VALUE_ONLY, USE_WRAPPER)
Review Comment:
I understand you're suggesting a different name. What would that be?
option:
- WRITE_VALUE, WRITE_WRAPPER
Or are you saying there shouldn't be an option?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
