hi Gaurav Thanks for this KIP as I totally agree that `cause` can offer more useful information. Please take a look at following small questions.
chia_0: do we need to deprecate existent constructor? It seems to me that is not conflict to the new one. chia_1: Have you consider adding `ConfigException(String name, Object value, String message, Throwable cause)`? it is used widely too. thanks, Chia-Ping On 2025/01/26 11:08:59 Gaurav Narula wrote: > Hi Everyone, > > I'd like to initiate a discussion on KIP-1129: Update ConfigException > constructors at https://cwiki.apache.org/confluence/x/n4pEF. > > This KIP adds a constructor to ConfigException to accept a Throwable as a > second argument following the Java convention for Throwables and deprecates > the existing constructor that accepts an Object as the second argument. > > Looking forward to hearing the community's feedback on this! > > Regards, > Gaurav
