diqiu50 commented on code in PR #11189:
URL: https://github.com/apache/gravitino/pull/11189#discussion_r3304140776
##########
flink-connector/flink-common/src/main/java/org/apache/gravitino/flink/connector/hive/GravitinoHiveCatalog.java:
##########
@@ -170,9 +186,7 @@ public void alterTable(ObjectPath tablePath,
CatalogBaseTable newTable, boolean
if (!(newTable instanceof ResolvedCatalogTable)) {
throw new CatalogException("Generic table must be a resolved catalog
table");
}
- // For generic tables, we re-serialize the entire table schema and
partition keys into
- // flink.* properties, so the individual tableChanges are not needed. The
newTable
- // parameter contains the final state after applying all changes.
+ // For generic tables, re-serialize the entire schema into flink.*
properties.
Review Comment:
fixed
--
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]
