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

    https://github.com/apache/nifi/pull/2447#discussion_r165436273
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java
 ---
    @@ -123,8 +124,22 @@
         public static final PropertyDescriptor MAX_ROWS_PER_FLOW_FILE = new 
PropertyDescriptor.Builder()
                 .name("qdbt-max-rows")
                 .displayName("Max Rows Per Flow File")
    -            .description("The maximum number of result rows that will be 
included in a single FlowFile. " +
    -                    "This will allow you to break up very large result 
sets into multiple FlowFiles. If the value specified is zero, then all rows are 
returned in a single FlowFile.")
    +            .description("The maximum number of result rows that will be 
included in a single FlowFile. This will allow you to break up very large "
    +                    + "result sets into multiple FlowFiles. If the value 
specified is zero, then all rows are returned in a single FlowFile.")
    +            .defaultValue("0")
    --- End diff --
    
    When I did something similar on GetMongo, I think I chose to make it 
optional and have "blank" be the equivalent. Thoughts?


---

Reply via email to