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

Reply via email to