Skip to content

[test](regression) Move Iceberg REST HDFS case to P2#63367

Open
xylaaaaa wants to merge 1 commit into
apache:masterfrom
xylaaaaa:codex/move-iceberg-rest-hdfs-p2
Open

[test](regression) Move Iceberg REST HDFS case to P2#63367
xylaaaaa wants to merge 1 commit into
apache:masterfrom
xylaaaaa:codex/move-iceberg-rest-hdfs-p2

Conversation

@xylaaaaa
Copy link
Copy Markdown
Contributor

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #60915

Problem Summary: Move iceberg_rest_on_hdfs out of the P0 external suite because it requires the dedicated iceberg-rest Docker environment, which is not started by the default community external pipeline.

Release note

None

Check List (For Author)

  • Test: Manual test
    • Verified the suite is declared only under external_table_p2 with group p2,external
    • Ran git diff --check
    • Regression test not run because the case requires the dedicated iceberg-rest Docker environment
  • Behavior changed: No
  • Does this need documentation: No

### What problem does this PR solve?

Issue Number: close #xxx

Related PR: apache#60915

Problem Summary: Move iceberg_rest_on_hdfs out of the P0 external suite because it requires the dedicated iceberg-rest Docker environment, which is not started by the default community external pipeline.

### Release note

None

### Check List (For Author)

- Test: Manual test
    - Verified the suite is declared only under external_table_p2 with group p2,external
    - Ran git diff --check
    - Regression test not run because the case requires the dedicated iceberg-rest Docker environment
- Behavior changed: No
- Does this need documentation: No
Copilot AI review requested due to automatic review settings May 18, 2026 12:05
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 regression test suite classification for iceberg_rest_on_hdfs, effectively changing when/where it runs in the external table test pipeline.

Changes:

  • Downgraded the suite tag from p0 to p2 for iceberg_rest_on_hdfs.

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

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

2 participants