apache / apache/polaris

Re-Enable ViewCatalogTests for cloudTests

Closed
#3,174 3 comments 0 reactions 0 assignees View on GitHub
enhancement stale
Dominant language
Java
Stars
2.1k
Forks
522
Avg merge
2d 1h
Merged PRs (30d)
140

Description

### Is your feature request related to a problem? Please describe.

Some tests have been disabled in https://github.com/apache/polaris/pull/3095, because ViewCatalogTest was depending on `@TempDir`.

### Describe the solution you'd like

https://github.com/apache/iceberg/pull/14653 will allow parameterizing ViewCatalogTests, once Polaris is on a version that has this fix included, we should re-enable these (https://github.com/apache/polaris/pull/3095) tests.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing Apache Polaris pull request 3095 and Apache Iceberg pull request 14653 to confirm the parameterization fix and identify the disabled ViewCatalogTests. Check whether Polaris uses a version containing the Iceberg fix, then re-enable the tests and verify the cloudTests suite passes.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
testing
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.