RocMarshal commented on code in PR #1: URL: https://github.com/apache/flink-connector-http/pull/1#discussion_r2365694816
########## flink-connector-http/src/main/java/org/apache/flink/connector/http/utils/ExceptionUtils.java: ########## Review Comment: I'm curious why the existing `org.apache.flink.util.ExceptionUtils.stringifyException` wasn't used here, and instead a new class was introduced? For example: To reduce dependencies. Or considering that in the future, it's highly likely that other methods will be added to this class, making it incompatible for sharing with Flink's existing ExceptionUtils. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
