Zouxxyy commented on code in PR #3843:
URL: https://github.com/apache/paimon/pull/3843#discussion_r1849414511
##########
paimon-core/src/main/java/org/apache/paimon/catalog/FileSystemCatalog.java:
##########
@@ -117,6 +119,9 @@ protected void dropTableImpl(Identifier identifier) {
@Override
public void createTableImpl(Identifier identifier, Schema schema) {
+ checkArgument(
+ !schema.options().containsKey(CoreOptions.PATH.key()),
Review Comment:
> Therefore, instead of relaxing the checks here, I suggest we optimize the
documentation by clearly stating this restriction in the documentation. We can
declare the location management restrictions of `FileSystemCatalog` and the
recommended usage of Spark DDL. What do you think?
I'm +1 for this
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
