accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [accumulo] ctubbsii commented on issue #1176: Update to commons-configuration2.5
Date Mon, 03 Jun 2019 03:53:58 GMT
ctubbsii commented on issue #1176: Update to commons-configuration2.5
URL: https://github.com/apache/accumulo/pull/1176#issuecomment-498104270
 
 
   I may be able to use the `PropertiesConfigurationLayout` directly in some places to read/write
the properties files instead of the `FileBasedConfigurationBuilder`. Doing so may help avoid
the extra optional dependency on commons-beanutils. However, I'm not sure if it will have
the same property interpolation features, so I'd need to check that first.
   
   Two other things that I should do before merging: check LICENSE/NOTICE change implications
for the updated dependencies, and possibly remove the custom `GENERAL_DYNAMIC_CLASSPATHS`
property interpolation feature, which isn't really needed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message