airborne12 commented on PR #67363:
URL: https://github.com/apache/doris/pull/67363#issuecomment-5503079294

   Correction to the reasoning in this PR's description: I wrote that 
branch-4.1 "lacks master's prevent_search_row_fallback path (added by #66052), 
so on this branch a search that does not take the index surfaces the internal 
error directly instead of falling back". That is wrong. 
`prevent_search_row_fallback` converts fallback-eligible failures into hard 
errors — it does not let a failed index lookup succeed — and it is not on this 
code path at all. The failing path, `VSearchExpr::execute_column_impl` 
returning `SearchExpr should not be executed without inverted index` when 
`fast_execute()` finds no usable index, is identical on master and branch-4.1.
   
   So master is exposed to the same flakiness, and I have opened #67402 to 
apply the same nonConcurrent grouping there. Everything else in the description 
— the measured 2-of-4 failure rate on branch-4.1, the local 1-BE/3-BE runs 
passing, and the fix itself — is unchanged.


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