airborne12 commented on PR #66876: URL: https://github.com/apache/doris/pull/66876#issuecomment-5325052263
The failed BE UTs were stale expectations exposed by this PR, not a production regression. They asserted that SNII could consult a raw-query cache entry even when the request provider had no complete immutable analyzer identity, which is exactly the cross-generation reuse this PR removes. The revised test preloads the identical raw query, then verifies that incomplete/no-identity providers bypass that entry, enter the segment, and invoke their own analyzer. Red: the three reported tests failed locally with the old expectations. Green: `InvertedIndexReaderAnalysisPurposeTest.*:SniiIndexReaderCountFallback.*` passes 37/37 under ASAN, and `./build.sh --be -j 192` completes successfully. Fixed in `0c7cd3b4c7e`. -- 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]
