TyrantLucifer commented on code in PR #3581: URL: https://github.com/apache/incubator-seatunnel/pull/3581#discussion_r1032861910
########## seatunnel-connectors-v2/connector-hudi/src/main/java/org/apache/seatunnel/connectors/seatunnel/hudi/exception/HudiConnectorErrorCode.java: ########## @@ -17,13 +17,28 @@ package org.apache.seatunnel.connectors.seatunnel.hudi.exception; -public class HudiPluginException extends Exception{ +import org.apache.seatunnel.common.exception.SeaTunnelErrorCode; - public HudiPluginException(String message) { - super(message); +public enum HudiConnectorErrorCode implements SeaTunnelErrorCode { + + CREATE_PARQUET_METADATA_FAILED("HUDI-01", "Create ParquetMetadata failed."), + KERBEROS_AUTHORIZED_FAILED("HUDI-02", "Kerberos Authorized failed"); Review Comment: How about add this error code to common? I think it can be reused in many connectors -- 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: commits-unsubscr...@seatunnel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org