remove Catalog.ColumnAlreadyExistException when apply applyAddColumnE…

…ventWithPosition in paimon. because use may create table already before with the target column.
apache · Jan 7, 2025 · 9842c50 · 9842c50
1 parent f91f167
commit 9842c50
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/...imon/src/main/java/org/apache/flink/cdc/connectors/paimon/sink/PaimonMetadataApplier.java b/...imon/src/main/java/org/apache/flink/cdc/connectors/paimon/sink/PaimonMetadataApplier.java
@@ -201,7 +201,11 @@ private void applyAddColumn(AddColumnEvent event) throws SchemaEvolveException {
         } catch (Catalog.TableNotExistException
                 | Catalog.ColumnAlreadyExistException
                 | Catalog.ColumnNotExistException e) {
-            throw new SchemaEvolveException(event, e.getMessage(), e);
+            if (e instanceof Catalog.ColumnAlreadyExistException) {
+                LOG.warn("{}, skip it.", e.getMessage());
+            } else {
+                throw new SchemaEvolveException(event, e.getMessage(), e);
+            }
         }
     }
 
@@ -293,11 +297,7 @@ private void applyDropColumn(DropColumnEvent event) throws SchemaEvolveException
         } catch (Catalog.TableNotExistException
                 | Catalog.ColumnAlreadyExistException
                 | Catalog.ColumnNotExistException e) {
-            if (e instanceof Catalog.ColumnAlreadyExistException) {
-                LOG.warn("{}, skip it.", e.getMessage());
-            } else {
-                throw new SchemaEvolveException(event, e.getMessage(), e);
-            }
+            throw new SchemaEvolveException(event, e.getMessage(), e);
         }
     }