Gabriel39 commented on code in PR #66599:
URL: https://github.com/apache/doris/pull/66599#discussion_r3747081988
##########
fe/fe-core/src/main/java/org/apache/doris/datasource/ExternalDatabase.java:
##########
@@ -792,6 +792,9 @@ public CatalogIf getCatalog() {
public boolean registerTable(TableIf tableIf) {
makeSureInitialized();
T table = (T) tableIf;
+ // Replayed metadata may lose these non-persistent owner references,
so restore them before publication.
Review Comment:
Thanks, agreed. I removed the event-path ownership mutation and the
synthetic field-clearing test. The PR now keeps only the defensive MTMV guard.
I also added an image round-trip test showing that external object caches are
excluded and lazy loading rebuilds the table with its catalog and database
before BaseTableInfo construction.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
