Skip to content

[fix](regression) Enable Paimon GCS regression#63521

Open
xylaaaaa wants to merge 1 commit into
apache:masterfrom
xylaaaaa:fix/paimon-gcs-regression
Open

[fix](regression) Enable Paimon GCS regression#63521
xylaaaaa wants to merge 1 commit into
apache:masterfrom
xylaaaaa:fix/paimon-gcs-regression

Conversation

@xylaaaaa
Copy link
Copy Markdown
Contributor

@xylaaaaa xylaaaaa commented May 22, 2026

What problem does this PR solve?

Issue Number: None

Related PR: None

Problem Summary: Re-enable test_paimon_gcs in the external regression pipeline and explicitly mark the Paimon catalog storage as GCS so Doris does not infer it as another S3-compatible provider. The case continues to use the existing S3-compatible warehouse path for GCS because the FE pipeline classpath does not provide native gs:// Paimon FileIO support.

Release note

None

Check List (For Author)

  • Test: Regression test / Unit Test
    • Regression test: ./run-regression-test.sh --run --conf regression-test/pipeline/external/conf/regression-conf.groovy -d external_table_p0/paimon -s test_paimon_gcs -dryRun
    • Unit Test: ./run-fe-ut.sh --run org.apache.doris.datasource.property.storage.GCSPropertiesTest
  • Behavior changed: No
  • Does this need documentation: No

Copilot AI review requested due to automatic review settings May 22, 2026 05:48
@hello-stephen
Copy link
Copy Markdown
Contributor

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?

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the Paimon GCS regression test to use the correct GCS warehouse scheme and enables the suite to run in the external regression pipeline.

Changes:

  • Switch warehouse URI from s3:// to gs:// in the Paimon GCS test suite.
  • Add fs.gcs.support = true to the relevant CREATE CATALOG definitions.
  • Remove test_paimon_gcs from the external regression excludeSuites list.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
regression-test/suites/external_table_p0/paimon/test_paimon_gcs.groovy Uses gs:// warehouse and enables GCS filesystem support in catalog properties.
regression-test/pipeline/external/conf/regression-conf.groovy Allows test_paimon_gcs to run by removing it from exclusions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@xylaaaaa
Copy link
Copy Markdown
Contributor Author

run buildall

### What problem does this PR solve?

Issue Number: None

Related PR: None

Problem Summary: Re-enable test_paimon_gcs in the external regression pipeline and configure the Paimon catalog to use the GCS filesystem scheme explicitly.

### Release note

None

### Check List (For Author)

- Test: Regression test / Unit Test / Manual test
    - Regression test: ./run-regression-test.sh --run --conf regression-test/pipeline/external/conf/regression-conf.groovy -d external_table_p0/paimon -s test_paimon_gcs -dryRun
    - Unit Test: ./run-fe-ut.sh --run org.apache.doris.datasource.property.storage.GCSPropertiesTest
    - Manual test: Attempted ./run-regression-test.sh --run --conf regression-test/pipeline/external/conf/regression-conf.groovy -d external_table_p0/paimon -s test_paimon_gcs; local FE at 172.19.0.2:9131 was unreachable before SQL execution.
- Behavior changed: No
- Does this need documentation: No
@xylaaaaa xylaaaaa force-pushed the fix/paimon-gcs-regression branch from bb6a10e to afb861c Compare May 22, 2026 09:32
@xylaaaaa
Copy link
Copy Markdown
Contributor Author

run buildall

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.

3 participants