andrew4699 commented on code in PR #922:
URL: https://github.com/apache/polaris/pull/922#discussion_r2003820853
##########
service/common/src/main/java/org/apache/polaris/service/catalog/BasePolarisCatalog.java:
##########
@@ -1229,11 +1243,14 @@ public void doRefresh() {
Set.of(PolarisStorageActions.READ));
return TableMetadataParser.read(fileIO, metadataLocation);
});
+ polarisEventListener.onAfterTableRefreshed(new
AfterTableRefreshedEvent(tableIdentifier));
Review Comment:
"Because it's possible" minimizes the argument. The events have concrete use
cases, but it's impossible to know all concrete use cases, so it's an effort by
everyone to implement what they need. If it's helpful I can list a concrete use
case per event, but also it should be possible as a self-exercise.
--
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-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
