mchades commented on code in PR #12565:
URL: https://github.com/apache/gravitino/pull/12565#discussion_r3853588518
##########
core/src/main/java/org/apache/gravitino/GravitinoEnv.java:
##########
@@ -869,6 +883,15 @@ private void initGravitinoServerComponents() {
new ViewEventDispatcher(eventBus, viewNormalizeDispatcher);
this.viewDispatcher = viewEventDispatcher;
+ // Semantic Model operation chain: SemanticModelNormalizeDispatcher ->
+ // SemanticModelOperationDispatcher -> ManagedSemanticModelOperations.
+ // TODO: Add event and hook layers with Semantic Model server integration.
Review Comment:
Agreed. This is a framework-only PR and does not expose Semantic Model
operations through the server or REST layer. #12595 tracks event and audit
support and must land before server integration, while #12594 separately tracks
ownership and privilege hooks. I’ve updated the TODOs to reference both
follow-up issues.
--
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]
