github-actions[bot] commented on code in PR #68170:
URL: https://github.com/apache/doris/pull/68170#discussion_r4063448245
##########
fe/fe-core/src/main/java/org/apache/doris/mtmv/ivm/IvmFailureReason.java:
##########
@@ -38,6 +38,7 @@ public enum IvmFailureReason {
public boolean requiresCompleteRefresh() {
return this == BINLOG_BROKEN
+ || this == STREAM_UNSUPPORTED
Review Comment:
Rechecked on current head `0148150244cb2103adaa88ac12df5cd6737cfd54`: the
later partition-scope change re-exposes the pending-partial-baseline case from
this thread. `PARTITIONS FALLBACK` now builds `[PARTITIONS, COMPLETE]`, while
the early stream check only runs when the list contains `IVM`. If a partial
baseline barrier is already pending, `handlePendingIvmBaselineRebuild` calls
the low-level partition refresh directly before the new scoped preflight; a
missing non-PCT stream (or mapped PCT stream) throws there, so the attempt loop
never reaches `COMPLETE`. Please apply the selected-partition stream check to
that pending rebuild before writing its guard, route fallback-authorized
requests to `COMPLETE`, and cover a non-empty pending partial barrier plus a
missing stream.
--
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]
