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

    https://github.com/apache/nifi/pull/3133#discussion_r232011596
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-services/nifi-dbcp-service-bundle/nifi-dbcp-service/src/main/java/org/apache/nifi/dbcp/DBCPConnectionPool.java
 ---
    @@ -164,6 +166,77 @@ public ValidationResult validate(final String subject, 
final String input, final
             
.expressionLanguageSupported(ExpressionLanguageScope.VARIABLE_REGISTRY)
             .build();
     
    +    public static final PropertyDescriptor MIN_IDLE = new 
PropertyDescriptor.Builder()
    +            .displayName("Minimum Idle Connections")
    +            .name("dbcp-mim-idle-conns")
    +            .description("The minimum number of connections that can 
remain idle in the pool, without extra ones being " +
    +                    "created, or zero to create none.")
    +            
.defaultValue(String.valueOf(GenericObjectPoolConfig.DEFAULT_MIN_IDLE))
    --- End diff --
    
    I didn't realize the constants were only available from a class in an 
"impl" package in DBCP. In that case I take back what I said :(  Can we create 
our own constants here with a comment that we got the values from DBCP itself?


---

Reply via email to