Use a FileHandler when loading properties

[ddanielr]

Fixes error when using include or includeOptional property directives in accumulo.properties

Caused by: org.apache.commons.configuration2.ex.ConfigurationException: Load operation not properly initialized! Do not call read(InputStream) directly, but use a FileHandler to load a configuration.
	at org.apache.commons.configuration2.PropertiesConfiguration.loadIncludeFile(PropertiesConfiguration.java:1395)
	at org.apache.commons.configuration2.PropertiesConfiguration.propertyLoaded(PropertiesConfiguration.java:1479)
	at org.apache.commons.configuration2.PropertiesConfigurationLayout.load(PropertiesConfigurationLayout.java:444)
	at org.apache.commons.configuration2.PropertiesConfiguration.read(PropertiesConfiguration.java:1500)
	at org.apache.accumulo.start.classloader.AccumuloClassLoader.getAccumuloProperty(AccumuloClassLoader.java:94)

Discovered when testing #5231, fixes #5234

[keith-turner]

That was a nice error message that pointed directly to FileHandler. Wonder if FileHandler should be used for the client properties.

[keith-turner]

The code for loading client props seems to use java Properties and not commons config, so FileHandler would not be applicable there.

accumulo/core/src/main/java/org/apache/accumulo/core/clientImpl/ClientInfoImpl.java

Line 113 in cf42adf

public static Properties toProperties(Path propertiesFile) {