thenatog commented on code in PR #6649:
URL: https://github.com/apache/nifi/pull/6649#discussion_r1024497984
##########
nifi-nar-bundles/nifi-amqp-bundle/nifi-amqp-processors/src/main/java/org/apache/nifi/amqp/processors/PublishAMQP.java:
##########
@@ -108,10 +106,22 @@ public class PublishAMQP extends
AbstractAMQPProcessor<AMQPPublisher> {
.addValidator(StandardValidators.SINGLE_CHAR_VALIDATOR)
.required(false)
.build();
+
+ public static final PropertyDescriptor IGNORE_HEADER_WITH_NULL_VALUE = new
PropertyDescriptor.Builder()
Review Comment:
Currently with 'Ignore Null Header = true', the processor throws an error,
which is a little counter-intuitive because we told the processor to ignore the
null header but it throws a warning?
Perhaps this would be better as 'Allow Null Header', and flipping the
true/false condition, such that the warning message is thrown when 'Allow Null
Header = false' and a null header is provided.
--
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]
