Skip to content

[fix](regression) Mirror HTTP TVF fixtures in regression OSS#66013

Merged
yiguolei merged 1 commit into
apache:branch-4.1from
hello-stephen:agent/branch41-http-tvf-hf
Jul 24, 2026
Merged

[fix](regression) Mirror HTTP TVF fixtures in regression OSS#66013
yiguolei merged 1 commit into
apache:branch-4.1from
hello-stephen:agent/branch41-http-tvf-hf

Conversation

@hello-stephen

Copy link
Copy Markdown
Contributor

What

Replace the mutable/public HuggingFace URLs in external_table_p0/tvf/test_http_tvf with stable public mirrors in the Doris regression OSS bucket.

Why

The branch-4.1 S3 P2 validation reached the HuggingFace section and failed with java.net.SocketTimeoutException: Connect timed out. The test data should not depend on HuggingFace network availability or mutable dataset branches.

The prompts fixture is pinned to source revision 61229c8c07ea12c6e219b2b2b728f23f2842901b, which preserves the expected 204 records. The IMDb JSON and Parquet fixtures are mirrored byte-for-byte. Existing golden output is unchanged.

Jira: DORIS-27400

Validation

  • git diff --check
  • Groovy 4.0.19 compilation
  • Suite load/discovery: test_http_tvf, tag p2
  • Public OSS GET and byte-range GET for all four fixtures
  • Local/remote MD5 matches for all fixtures
  • Prompts CSV logical record count: 204

This is a P2 case-only change; after merge it still requires an exact post-merge rerun of external_table_p0/tvf/test_http_tvf.

@hello-stephen

Copy link
Copy Markdown
Contributor Author

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@hello-stephen

Copy link
Copy Markdown
Contributor Author

[doris-27400-static-gate]

This is a P2 case-only fix. It does not touch the regression framework, shared utilities, dependencies, or build paths.

Validated before requesting the CI skip:

  • git diff --check
  • Groovy 4.0.19 compilation
  • suite load/discovery: test_http_tvf, tag p2
  • all four mirrored objects are publicly readable and support byte-range GET (206)
  • local/remote MD5 matches for every fixture
  • pinned prompts snapshot contains exactly the expected 204 logical records
  • existing golden output is unchanged

Community buildall does not execute this P2 case. The required proof remains an exact post-merge branch-4.1 rerun of external_table_p0/tvf/test_http_tvf.

@hello-stephen

Copy link
Copy Markdown
Contributor Author

skip buildall

@hello-stephen
hello-stephen marked this pull request as ready for review July 24, 2026 10:41
@hello-stephen
hello-stephen requested a review from yiguolei as a code owner July 24, 2026 10:41
@hello-stephen

Copy link
Copy Markdown
Contributor Author

/review

@github-actions

Copy link
Copy Markdown
Contributor

Codex automated review failed and did not complete.

Error: You've hit your usage limit. Visit https://chatgpt.com/codex/settings/usage to purchase more credits or try again at Jul 30th, 2026 7:58 AM.
Workflow run: https://github.com/apache/doris/actions/runs/30087052068

Please inspect the workflow logs and rerun the review after the underlying issue is resolved.

@yiguolei
yiguolei merged commit 1ac74c8 into apache:branch-4.1 Jul 24, 2026
32 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants