924060929 commented on code in PR #69806:
URL: https://github.com/apache/doris/pull/69806#discussion_r4439789709


##########
fe/fe-connector/fe-connector-iceberg/src/main/java/org/apache/doris/connector/iceberg/IcebergExceptionUtils.java:
##########
@@ -31,12 +33,25 @@ private IcebergExceptionUtils() {
 
     static RuntimeException wrapTableLoadFailure(
             IcebergTableHandle handle, Exception failure, String 
fallbackPrefix) {
+        if (isTableAbsent(failure)) {

Review Comment:
   Final rebased head: 0bc986e0d097, based on master49bfbdecffb. The actual 
conflict was in resolveConnectorTableHandle: master's view classification must 
precede the table endpoint for connectors returning non-404 view errors. It is 
preserved inside the existing guarded metadata read. Added a regression that 
retains a live cached view without calling that endpoint, then verifies 
confirmed view deletion still cleans the stale identity. The partition-name, 
metadata-only replacement, and cold replay repairs above are retained; 
range-diff confirms the other nine original patches are unchanged.
   
   Validation: 1025 focused unit cases passed across 51 classes; FE build and 
Checkstyle passed. Controlled providers, not a live external-service 
integration run.



-- 
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]

Reply via email to