924060929 commented on code in PR #69629:
URL: https://github.com/apache/doris/pull/69629#discussion_r4385014482
##########
fe/fe-core/src/main/java/org/apache/doris/datasource/iceberg/IcebergExternalTable.java:
##########
@@ -366,8 +379,10 @@ public Map<String, SysTable> getSupportedSysTables() {
@Override
public boolean isView() {
- makeSureInitialized();
- return isView;
+ return withMissingTableRepair(() -> {
Review Comment:
Fixed in 02879aaeb93f. getViewText() and getSqlDialect() now use the common
guarded getIcebergView() load. The SELECT binding view-text entry and the real
SHOW CREATE helper both repair a confirmed NoSuchViewException; the dialect
path is covered too. Three tests reproduced zero repairs before this fix. A
real MetaCache interleaving test publishes a same-name replacement before the
old request returns absence and verifies the new object and numeric ID survive.
Validation: 92 selected FE unit tests, FE build and Checkstyle passed. SDK
absence is mocked; this is not a live REST/SQL integration claim.
--
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]
