Skip to content

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

Merged
Gabriel39 merged 1 commit into
apache:masterfrom
xylaaaaa:codex/move-iceberg-rest-hdfs-p2
May 19, 2026
Merged

[test](regression) Move Iceberg REST HDFS case to P2#63367
Gabriel39 merged 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

@Gabriel39
Copy link
Copy Markdown
Contributor

/review

@github-actions github-actions Bot added the approved Indicates a PR has been approved by one committer. label May 19, 2026
@github-actions
Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actions
Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@Gabriel39 Gabriel39 merged commit cbffd4f into apache:master May 19, 2026
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. dev/4.1.x dev/4.1.x-conflict reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants