Gabriel39 commented on PR #69672: URL: https://github.com/apache/doris/pull/69672#issuecomment-6464362353
Rechecked all nine inline review threads, including follow-ups inside already-resolved threads, and the complete review summaries. Updated in 789ceff51d1bfcaff7de1da02f9d8d18d715846e. - Completed the two remaining instances: supported doc-value-only numeric/date/boolean/IP queries, and configuration-only alias backings in either response order. - Preserved the prior native-residual, source-array, source-filter, selected-path, sibling-mode and rounded-number fixes. The older scaled_float thread is now explicitly closed with its attribution correction. - Added the NULL existence guard needed by inequalities produced by native NOT rewrites; the new runtime oracle keeps NULL rows. Validation: 152 FE tests passed (one optional ES 9 compatibility class skipped), all nine reactor modules passed Checkstyle, and three live ES 8.11.3 suites passed 1,148 stored checks, 1,052 independent comparisons and 88 expected planning errors. All 876 existing stored results remain unchanged; the 272 new results were generated by the repository runner and independently rerun without generation. The PR description now explains the final execution flow and correctness/performance tradeoffs. -- 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]
