C0urante commented on PR #14721: URL: https://github.com/apache/kafka/pull/14721#issuecomment-1804080696
IMO it'd be cleaner if we could keep the same exception type. We could concatenate the existing messages with the cause's message (if non-null); for example: ```java String message = "A client SSLEngine created with the provided settings " + "can't connect to a server SSLEngine created with those settings." if (e.getMessage() != null) message += ": " + e.getMessage(); throw new ConfigException(message); ``` A follow-up item (that requires a KIP but hopefully shouldn't be too controversial) could be to add a new `ConfigException(String message, Throwable cause)` constructor, which would allow us to preserve the complete stack trace of the cause instead of just the message. -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org