Gabriel39 commented on PR #69672: URL: https://github.com/apache/doris/pull/69672#issuecomment-6468383364
Addressed the three new P2 findings in bbe765a93d950f19fbd7a73b494ab2917b52783b and rechecked the review inventory: all 21 threads are currently resolved. - Shared scalar source-array handling now covers floating-point, Boolean, integer and date singleton/empty/null-only arrays while preserving STRING and declared ARRAY containers. - Narrow integer strings follow double rounding and the ES family-specific bound order; long precision remains exact. - Retained range NOT preserves the original ES query complement with an existence guard, avoiding inverse-range matches on threshold-spanning arrays while retaining the prior NULL/equality fixes. Local validation passed: 159 FE tests (one optional ES9 class skipped), 17 BE unit tests, standard ASAN BE build, Checkstyle, clang-format 16, scoped clang-tidy and build hygiene. Five live ES8 suites passed 1,612 stored checks and 1,516 independent comparisons, preserving 88 expected planning errors. All 1,452 existing stored results are unchanged; the 160 new results were generated and then verified in a separate run without generation. The PR description now reflects the final execution flow and tests. Posted run buildall followed by /review after pushing; those new CI/review runs are pending. -- 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]
