yuqi1129 commented on code in PR #10393:
URL: https://github.com/apache/gravitino/pull/10393#discussion_r2958938820
##########
core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java:
##########
@@ -274,6 +274,10 @@ public Schema alterSchema(NameIdentifier ident,
SchemaChange... changes)
.withId(schemaEntity.id())
.withName(schemaEntity.name())
.withNamespace(ident.namespace())
+ .withComment(alteredSchema.comment())
+ .withProperties(
Review Comment:
It seems that this is by design, `comment` and `properties` will be provided
by `alteredSchema` in `EntityCombinedSchema`. This is to say: these two values
are decided by corresponding values in the underlying storage, not the
Gravitino store.
@jerryshao
Can you help to verify it?
--
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]
