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

    https://github.com/apache/nifi/pull/2162#discussion_r163454074
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/AbstractDatabaseFetchProcessor.java
 ---
    @@ -156,10 +157,22 @@
                 .addValidator(StandardValidators.NON_EMPTY_VALIDATOR)
                 .build();
     
    +    public static final PropertyDescriptor SQL_QUERY = new 
PropertyDescriptor.Builder()
    +            .name("db-fetch-sql-query")
    +            .displayName("Custom Query")
    +            .description("A custom SQL query used to retrieve data. 
Instead of building a SQL query from "
    +                    + "other properties, this query will be used. Query 
must have no WHERE or ORDER BY statements. "
    +                    + "If a WHERE clause is needed use a sub-query or the 
'Additional WHERE clause' property.")
    --- End diff --
    
    Sounds like a good idea. Users still won't be able to include an ORDER BY, 
as those are not allowed in sub queries in every database I've worked with.


---

Reply via email to