keith-turner commented on a change in pull request #1077: Updated commons
config from ver 1 to 2
URL: https://github.com/apache/fluo/pull/1077#discussion_r313617283
##########
File path:
modules/api/src/main/java/org/apache/fluo/api/config/SimpleConfiguration.java
##########
@@ -192,34 +195,30 @@ public void load(InputStream in) {
* @since 1.2.0
*/
public void load(File file) {
- try {
- PropertiesConfiguration config = new PropertiesConfiguration();
- // disabled to prevent accumulo classpath value from being shortened
- config.setDelimiterParsingDisabled(true);
- config.load(file);
+ try(InputStream in = Files.newInputStream(file.toPath())) {
+ PropertiesConfiguration config = newPropertiesConfiguration();
+ new FileHandler(config).load(cleanUp(in));
Review comment:
Done. When I actually when to use this I noticed I had to repeat the config
and found that a bit odd.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
