pinot icon indicating copy to clipboard operation
pinot copied to clipboard

Validate Duplicate Config Properties

Open gibbonjj opened this issue 2 years ago • 1 comments

The default behavior of PropertiesConfiguration when reading a .properties file is to create an array of the duplicate values for the key instead of a single string. This change throws a ConfigurationException if someone mistakenly adds duplicate keys.

Reference issue https://github.com/apache/pinot/issues/7882

gibbonjj avatar Mar 19 '24 02:03 gibbonjj

Thanks @gibbonjj for the contribution.

We recently updated commons-configuration1 package usage to commons-configuration2[issue reference].

I think in commons-configuration2 the default behaviour is to overwrite the property value in-case if it's already present. Can you please confirm the behaviour ? Thanks

abhioncbr avatar Mar 19 '24 05:03 abhioncbr