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

    https://github.com/apache/nifi/pull/2991#discussion_r218798942
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/HandleHttpRequest.java
 ---
    @@ -229,6 +252,14 @@
                 .name("container-queue-size").displayName("Container Queue 
Size")
                 .description("The size of the queue for Http Request 
Containers").required(true)
                 
.addValidator(StandardValidators.POSITIVE_INTEGER_VALIDATOR).defaultValue("50").build();
    +    public static final PropertyDescriptor MAX_REQUEST_SIZE = new 
PropertyDescriptor.Builder()
    +            .name("max-request-size")
    +            .displayName("Max Request Size")
    +            .description("The maximal size of the request")
    +            .required(true)
    +            .addValidator(StandardValidators.POSITIVE_LONG_VALIDATOR)
    --- End diff --
    
    This should be using a DATA_SIZE_VALIDATOR and use input in the format of 
"1 MB" instead of a Long value. Default value should be "1 MB" then.


---

Reply via email to