hawk9821 commented on code in PR #8074:
URL: https://github.com/apache/seatunnel/pull/8074#discussion_r1953958415
##########
seatunnel-connectors-v2/connector-paimon/src/main/java/org/apache/seatunnel/connectors/seatunnel/paimon/catalog/PaimonCatalog.java:
##########
@@ -216,7 +229,9 @@ private Schema buildPaimonSchema(@NonNull
org.apache.paimon.schema.TableSchema s
Schema.Builder builder = Schema.newBuilder();
schema.fields()
.forEach(field -> builder.column(field.name(), field.type(),
field.description()));
- builder.options(schema.options());
+ Map<String, String> options = new HashMap<>(schema.options());
+ options.remove(CoreOptions.PATH.key());
Review Comment:
The paimon interface has changed in the new version. it will throw
exception when not remove
```
java.lang.UnsupportedOperationException: The current catalog
FileSystemCatalog does not support specifying the table path when creating a
table.
at
org.apache.paimon.catalog.AbstractCatalog.validateCustomTablePath(AbstractCatalog.java:582)
at
org.apache.paimon.catalog.AbstractCatalog.createTable(AbstractCatalog.java:285)
at
org.apache.paimon.catalog.DelegateCatalog.createTable(DelegateCatalog.java:106)
```
--
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
