[ https://issues.apache.org/jira/browse/NIFI-1974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15322808#comment-15322808 ]
ASF GitHub Bot commented on NIFI-1974: -------------------------------------- Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/501#discussion_r66475348 --- Diff: nifi-api/src/main/java/org/apache/nifi/registry/FileVariableRegistry.java --- @@ -0,0 +1,70 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.apache.nifi.registry; + +import java.io.File; +import java.io.IOException; +import java.nio.file.Path; +import java.util.Map; + + +public abstract class FileVariableRegistry extends MultiMapVariableRegistry { + + public FileVariableRegistry() { + super(); + } + + public FileVariableRegistry(File... files){ + super(); + addVariables(files); + } + + public FileVariableRegistry(Path... paths){ + super(); + addVariables(paths); + } + + @SuppressWarnings({"unchecked", "rawtypes"}) + public void addVariables(File ...files){ + if(files != null) { + for (final File file : files) { + try { + registry.addMap(convertFile(file)); + } catch (IOException iex) { + throw new IllegalArgumentException("A file provided was invalid.", iex); --- End diff -- After reviewing I made the suggested change, since removing the public off of the constructor to ensure only Factory creation made the exception on the constructor much more palatable :). > Support Custom Properties in Expression Language > ------------------------------------------------ > > Key: NIFI-1974 > URL: https://issues.apache.org/jira/browse/NIFI-1974 > Project: Apache NiFi > Issue Type: New Feature > Reporter: Yolanda M. Davis > Assignee: Yolanda M. Davis > Fix For: 1.0.0 > > > Add a property in "nifi.properties" config file to allows users to specify a > list of custom properties files (containing data such as environmental > specific values, or sensitive values, etc.). The key/value pairs should be > loaded upon NIFI startup and availbale to processors for use in expression > languages. > Optimally this will lay the groundwork for a UI driven Variable Registry. -- This message was sent by Atlassian JIRA (v6.3.4#6332)