hello-stephen opened a new pull request, #70411:
URL: https://github.com/apache/doris/pull/70411

   ### Background
   
   The performance pipeline creates `clickbench.hits` in `run-clickbench.sh`. 
The load benchmark then copies ten million rows from that table into 
`load_test_db.hits_insert_into_select` and checks row count and throughput.
   
   ### What problem does this PR solve?
   
   Issue Number: None
   Related PR: None
   PR Type: Test
   
   Problem Summary:
   
   The INSERT SELECT target declares `EventDate`, `EventTime`, 
`ClientEventTime`, and `LocalEventTime` as `INT`, while the source uses 
`DateV2` and `DateTimeV2`. This makes the load benchmark depend on unintended 
date/time-to-integer conversions and can fail before measuring insertion 
throughput. Match the target types to the existing source schema.
   
   ### Details
   
   - **Failure:** The INSERT SELECT benchmark can fail on incompatible 
date/time-to-integer assignment before its row-count and throughput checks.
   - **Cause:** The target DDL uses integer types for four columns copied 
directly from the ClickBench table.
   - **Stability evidence:** Static comparison of all 105 ordered column 
declarations finds four mismatches before this change and none afterward. The 
ten-million-row assertion, load sequence, failure handling, and performance 
thresholds remain unchanged. JSON already uses date/time types; ORC and Parquet 
keep their fixture-specific schemas.
   - **Behavior change:** The transient INSERT SELECT target now stores 
date/time values using the source column types.
   
   ### Testing
   
   - Locally passed `bash -n regression-test/pipeline/performance/run-load.sh`.
   - Locally passed `git diff --check`.
   - Compared all 105 source and target column names, types, and nullability in 
order: four differences before, zero after.
   - Live SQL execution and the full performance pipeline have not run for this 
commit. This draft needs the original complete fixtures and a validated 
Performance Release build for runtime verification.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Behavior changed: Yes (see Details)
   - Does this need documentation: No
   - Pre-submission status: Not ready: draft pending live INSERT SELECT and 
full performance validation.
   
   ### AI Disclosure
   
   - Tool: OpenAI Codex
   - Generated: The four DDL type changes and this description.
   - Human review: 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]

Reply via email to