abhioncbr commented on PR #12672: URL: https://github.com/apache/pinot/pull/12672#issuecomment-2005830577
Thanks @gibbonjj for the contribution. We recently updated commons-configuration1 package usage to commons-configuration2[[issue reference](https://github.com/apache/pinot/issues/11085)]. I think in commons-configuration2 the [default behaviour is to overwrite](https://github.com/apache/commons-configuration/blob/master/src/main/java/org/apache/commons/configuration2/AbstractConfiguration.java#L662) the property value in-case if it's already present. Can you please confirm the behaviour ? Thanks -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
