morrySnow opened a new pull request, #61842:
URL: https://github.com/apache/doris/pull/61842

   ## Proposed changes
   
   Merge unique test content from `nereids_p0` regression tests into `query_p0` 
to eliminate duplication.
   
   ### What problem does this PR solve?
   
   Issue Number: close #xxx
   
   Problem Summary: The `regression-test/suites/nereids_p0/` folder was copied 
from `query_p0` approximately 4 years ago to develop the Nereids optimizer. Now 
that Nereids is the default optimizer, there are 119 files at the same relative 
path in both folders with overlapping test coverage. This PR merges all unique 
test content from `nereids_p0` into `query_p0` to consolidate these tests into 
a single location.
   
   **Scope of changes:**
   - 42 Groovy test files in `regression-test/suites/query_p0/` modified to 
include unique test cases from `nereids_p0`
   - 24 `.out` golden output files updated with expected results for newly 
added tests
   - 1 new `.out` file created (`session_variable/test_default_limit.out`)
   - All `enable_nereids_planner` settings removed from merged content (Nereids 
is now default)
   - All `nereids_test_query_db` references replaced with `test_query_db`
   - 77 files where `query_p0` already contained all content from `nereids_p0` 
(no changes needed)
   
   **Key large merges:**
   - `explain/test_pushdown_explain.groovy`: +391 lines of pushdown predicate 
tests
   - `sql_functions/spatial_functions/test_gis_function.groovy`: +354 lines of 
spatial function tests
   - `subquery/test_subquery.groovy`: +253 lines of subquery tests
   - `sql_functions/datetime_functions/test_date_function.groovy`: +190+ lines 
of datetime tests
   - `join/test_runtimefilter_on_datev2.groovy`: +112 lines of runtime filter 
tests
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test: No need to test - this is a test file reorganization, merging test 
content from nereids_p0 into query_p0. The merged tests themselves serve as the 
test.
   - Behavior changed: No
   - Does this need documentation: No


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