Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2160#discussion_r140265302
--- Diff:
nifi-nar-bundles/nifi-kudu-bundle/nifi-kudu-processors/src/main/java/org/apache/nifi/processors/kudu/AbstractKudu.java
---
@@ -124,11 +148,14 @@ public void OnScheduled(final ProcessContext context)
{
kuduTable = this.getKuduTable(kuduClient, tableName);
getLogger().debug("Kudu connection successfully
initialized");
}
+
--- End diff --
I know this is not part of this PR, but would not it make sense to allow
expression language on table name and Kudu masters? (not with an evaluation
against flow files but just against the variable registry and such in case
someone wants to externalize the values between environments). Same remark for
batch size?
---
