[ 
https://issues.apache.org/jira/browse/NIFI-413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14592051#comment-14592051
 ] 

ASF GitHub Bot commented on NIFI-413:
-------------------------------------

Github user markap14 commented on a diff in the pull request:

    https://github.com/apache/incubator-nifi/pull/55#discussion_r32748767
  
    --- Diff: 
nifi/nifi-nar-bundles/nifi-kafka-bundle/nifi-kafka-processors/src/main/java/org/apache/nifi/processors/kafka/PutKafka.java
 ---
    @@ -136,6 +136,68 @@
                 .addValidator(StandardValidators.NON_EMPTY_VALIDATOR)
                 .expressionLanguageSupported(false)
                 .build();
    +    public static final PropertyDescriptor PRODUCER_TYPE = new 
PropertyDescriptor.Builder()
    +            .name("Producer Type")
    +            .description("This parameter specifies whether the messages 
are sent asynchronously in a background thread."
    +                    + " Valid values are (1) async for asynchronous send 
and (2) sync for synchronous send."
    +                    + " By setting the producer to async we allow batching 
together of requests (which is great for throughput)"
    +                    + " but open the possibility of a failure of the 
client machine dropping unsent data.")
    +            .required(true)
    +            .allowableValues("sync", "async")
    +            .addValidator(StandardValidators.NON_EMPTY_VALIDATOR)
    +            .expressionLanguageSupported(false)
    +            .defaultValue("sync")
    +            .build();
    +        public static final PropertyDescriptor BATCH_NUM_MESSAGES = new 
PropertyDescriptor.Builder()
    +            .name("Async Message Batch Size (batch.num.messages)")
    +            .description("Used only if Producer Type is set to \"async\". 
The number of messages to send in one batch when using async mode."
    +                    + " The producer will wait until either this number of 
messages are ready"
    +                    + " to send or queue.buffer.max.ms is reached.")
    +            .required(true)
    +            .addValidator(StandardValidators.POSITIVE_INTEGER_VALIDATOR)
    +            .defaultValue("200").build();
    +        public static final PropertyDescriptor QUEUE_BUFFERING_MAX_MS = 
new PropertyDescriptor.Builder()
    --- End diff --
    
    I would recommend we change this property to be a Time Period 
(StandardValidators.TIME_PERIOD_VALIDATOR) with a default value of "5 secs"


> KafkaPut should support compression option
> ------------------------------------------
>
>                 Key: NIFI-413
>                 URL: https://issues.apache.org/jira/browse/NIFI-413
>             Project: Apache NiFi
>          Issue Type: Improvement
>            Reporter: Michael Braun
>            Priority: Minor
>              Labels: beginner, newbie
>             Fix For: 0.2.0
>
>
> Kafka supports compression out of the box (GZIP, Snappy with another 
> dependency) but the KafkaPut operator does not expose the option. It would be 
> nice if this was exposed in processor settings so processing could benefit 
> from compression.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to