[GitHub] [incubator-paimon] zhuangchong commented on a diff in pull request #704: [core] Improve the exception information when the changelog producer is `lookup` , the table has no primary key

["zhuangchong (via GitHub)" <gi...@apache.org>]

zhuangchong commented on code in PR #704:
URL: https://github.com/apache/incubator-paimon/pull/704#discussion_r1148720200


##########
paimon-core/src/main/java/org/apache/paimon/schema/SchemaValidation.java:
##########
@@ -107,7 +109,7 @@ public static void validateTableSchema(TableSchema schema) {
                             checkState(
                                     !SYSTEM_FIELD_NAMES.contains(f),
                                     String.format(
-                                            "Field name[%s] in schema cannot be exist in [%s]",
+                                            "Field name[%s] in schema cannot be exist in %s",

Review Comment:
   I just modified the printed exception information, 
   before modification: `Field name["_VALUE_COUNT"] in schema cannot be exist in [["_VALUE_COUNT", "_SEQUENCE_NUMBER", "_VALUE_KIND"]]` ,
    after modification: `Field name["_VALUE_COUNT"] in schema cannot be exist in ["_VALUE_COUNT", "_SEQUENCE_NUMBER", "_VALUE_KIND"]`
   
   



-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org