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

    https://github.com/apache/nifi/pull/2091#discussion_r133906727
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GenerateTableFetch.java
 ---
    @@ -112,6 +112,17 @@
                 
.addValidator(StandardValidators.NON_NEGATIVE_INTEGER_VALIDATOR)
                 .build();
     
    +    public static final PropertyDescriptor RIGHT_BOUND_WHERE = new 
PropertyDescriptor.Builder()
    --- End diff --
    
    On my side, I've tested it on MS SQL databases. I agree that people will 
likely enable this option in all cases and that's just adding a WHERE clause in 
the generated queries. I don't see any reason why this would not work / would 
break existing workflows. I'm fine with removing the property or setting it to 
true by default in case of unforeseen behaviours. What do you think?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to