markap14 commented on code in PR #6131:
URL: https://github.com/apache/nifi/pull/6131#discussion_r921422679
##########
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:
Yes, sorry, I was suggesting renaming it. Perhaps something like "Use
Content as Record Value" or "FLowFile Content as Record Value" or something
like that? Was just trying to convey that we aren't *only* writing the record
value, a key and headers may also be written.
--
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]
