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

    https://github.com/apache/nifi/pull/2695#discussion_r188988886
  
    --- Diff: 
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/nifi/processors/hive/SelectHiveQL.java
 ---
    @@ -113,6 +126,17 @@
                 
.expressionLanguageSupported(ExpressionLanguageScope.FLOWFILE_ATTRIBUTES)
                 .build();
     
    +    public static final PropertyDescriptor HIVEQL_POST_QUERY = new 
PropertyDescriptor.Builder()
    +            .name("hive-post-query")
    +            .displayName("HiveQL Post-Query")
    +            .description("HiveQL post-query to execute. 
Semicolon-delimited list of queries. "
    +                    + "Example: 'set tez.queue.name=default; set 
hive.exec.orc.split.strategy=HYBRID; set 
hive.exec.reducers.bytes.per.reducer=258998272'. "
    +                    + "Note, the results/outputs of these queries will be 
suppressed if successful executed.")
    +            .required(false)
    +            .addValidator(StandardValidators.NON_EMPTY_VALIDATOR)
    --- End diff --
    
    Could it make sense to have a custom validator here just to be sure the 
statements will be accepted by the driver? (would need to check if EL is used 
or not first). Not sure it's worth the trouble though...


---

Reply via email to