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

   ## Proposed changes
   
   Follow-up for DORIS-27400 after #66013 replaced unstable HuggingFace URLs 
with immutable OSS fixtures.
   
   Post-merge exact S3 P2 build #204256 reached the mirrored `prompts.csv`, 
proving that the original network timeout is gone, but `qt_sql15` failed with 
expected 204 / actual 205. The mirrored CSV is byte-identical to the pinned 
dataset revision we captured and contains one header record plus 204 data 
records. Direct HTTP CSV TVF reads the header as data, while the previous 
HuggingFace path exposed the 204 dataset rows.
   
   This case-only follow-up adds `skip_lines = 1` to the three `prompts.csv` 
HTTP TVF calls (`qt_sql15`, `qt_sql16`, and `qt_sql17`) so the mirrored path 
preserves the existing 204-row semantics and schema assertion.
   
   ## Validation
   
   - #204256 terminal first error: `sql15` expected 204, actual 205; the OSS 
URL was reachable and the target executed
   - Mirrored CSV: 205 physical CSV records = 1 header + 204 data records
   - `git diff --check`: passed
   - Groovy 4.0.19 compilation: passed
   - Suite discover/load identity: exactly one `test_http_tvf`, group `p2`; no 
framework/tool/build changes
   
   A post-merge exact `external_table_p0/tvf/test_http_tvf` run is still 
required before closing DORIS-27400.


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