[ 
https://issues.apache.org/jira/browse/NIFI-2574?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15424855#comment-15424855
 ] 

ASF GitHub Bot commented on NIFI-2574:
--------------------------------------

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

    https://github.com/apache/nifi/pull/872#discussion_r75158697
  
    --- Diff: 
nifi-commons/nifi-properties/src/main/java/org/apache/nifi/util/NiFiProperties.java
 ---
    @@ -29,17 +26,22 @@
     import java.nio.file.Paths;
     import java.util.ArrayList;
     import java.util.Arrays;
    +import java.util.Collections;
     import java.util.HashMap;
     import java.util.List;
     import java.util.Map;
     import java.util.Properties;
    -
    -public class NiFiProperties extends Properties {
    -
    -    private static final long serialVersionUID = 2119177359005492702L;
    -
    -    private static final Logger LOG = 
LoggerFactory.getLogger(NiFiProperties.class);
    -    private static NiFiProperties instance = null;
    +import java.util.Set;
    +
    +/**
    + * The NiFiProperties class holds all properties which are needed for 
various
    + * values to be available at runtime. It is strongly tied to the startup
    + * properties needed and is often refer to as the 'nifi.properties' file. 
The
    --- End diff --
    
    should be "and is often referred to"


> Update NiFiProperties usage to a context based accessor rather than static 
> singleton
> ------------------------------------------------------------------------------------
>
>                 Key: NIFI-2574
>                 URL: https://issues.apache.org/jira/browse/NIFI-2574
>             Project: Apache NiFi
>          Issue Type: Sub-task
>          Components: Configuration, Core Framework
>            Reporter: Joseph Witt
>            Assignee: Joseph Witt
>              Labels: migration
>             Fix For: 1.0.0
>
>
> This is a key step to move our longstanding odd static singleton usage of 
> NiFiProperties which enables a loader to exist which will encapsulate all 
> logic for dealing with encrypted properties as is the goal of the parent task.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to