[
https://issues.apache.org/jira/browse/NIFI-2407?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15406491#comment-15406491
]
ASF GitHub Bot commented on NIFI-2407:
--------------------------------------
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/778#discussion_r73405976
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/AbstractHadoopProcessor.java
---
@@ -191,9 +191,19 @@ public final void abstractOnScheduled(ProcessContext
context) throws IOException
HdfsResources resources = hdfsResources.get();
if (resources.getConfiguration() == null) {
String configResources =
context.getProperty(HADOOP_CONFIGURATION_RESOURCES).getValue();
- String dir =
context.getProperty(DIRECTORY_PROP_NAME).getValue();
- dir = dir == null ? "/" : dir;
- resources = resetHDFSResources(configResources, dir,
context);
+ final String dir;
+ final PropertyDescriptor directoryPropDescriptor =
context.getProperties().keySet().stream()
--- End diff --
Why replace a (fast) map lookup with an iteration over all keys?
> Add EL support to processors/controller services
> ------------------------------------------------
>
> Key: NIFI-2407
> URL: https://issues.apache.org/jira/browse/NIFI-2407
> Project: Apache NiFi
> Issue Type: Bug
> Components: Extensions
> Reporter: Jeff Storck
> Assignee: Jeff Storck
> Fix For: 1.0.0
>
>
> Add support for EL:
> * "Directory" property for GetHDFS and ListHDFS processors
> * "Destination Name" property for ConsumeJMS and PublishJMS processors
> * "MQ ConnectionFactory Implementation", "MQ client library path", "Broker
> URI" properties for the JMS Connection Factory Provider
> * "Database Connection URL", "Database Driver Class Name", "DB Driver jar
> url", "DB username", and "DB password" properties for the DBCP Connection Pool
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
