gaurav-narula commented on code in PR #15994:
URL: https://github.com/apache/kafka/pull/15994#discussion_r1606729442
##########
clients/src/main/java/org/apache/kafka/common/config/ConfigException.java:
##########
@@ -33,6 +33,10 @@ public ConfigException(String name, Object value) {
this(name, value, null);
}
+ public ConfigException(String message, Throwable cause) {
+ super(message, cause);
Review Comment:
Thanks for the suggestion. We're very restricted with what we can do within
a constructor unfortunately as call to `super()/this()` must be the first
statement if it's to be used. This is due to change with Java 22 with [JEP
447](https://openjdk.org/jeps/447) landing but I doubt we can add a conditional
at the moment.
--
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]
