Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding RESTCatalog based Spark Integ Test #11079

Open
1 of 3 tasks
haizhou-zhao opened this issue Sep 4, 2024 · 5 comments
Open
1 of 3 tasks

Adding RESTCatalog based Spark Integ Test #11079

haizhou-zhao opened this issue Sep 4, 2024 · 5 comments
Labels
improvement PR that improves existing functionality

Comments

@haizhou-zhao
Copy link
Contributor

Feature Request / Improvement

Current Spark smoke test runs on Hive Catalog, it would be nice if we run the same test cases on top of REST Catalog as well. Example: https://github.com/apache/iceberg/blob/main/spark/v3.5/spark-runtime/src/integration/java/org/apache/iceberg/spark/SmokeTest.java

Query engine

None

Willingness to contribute

  • I can contribute this improvement/feature independently
  • I would be willing to contribute this improvement/feature with guidance from the Iceberg community
  • I cannot contribute this improvement/feature at this time
@haizhou-zhao haizhou-zhao added the improvement PR that improves existing functionality label Sep 4, 2024
@haizhou-zhao
Copy link
Contributor Author

cc: @danielcweeks

As discussed offline, I think the we can build on top of RCK classes from #10908 for REST Catalog based spark integration tests.

@haizhou-zhao
Copy link
Contributor Author

Found several issues while attempting to enable integration test based on REST Catalog for Spark. Listing them below as I troubleshoot them one by one.

@haizhou-zhao
Copy link
Contributor Author

#11103

@haizhou-zhao
Copy link
Contributor Author

#11109

@haizhou-zhao
Copy link
Contributor Author

#11154

@haizhou-zhao haizhou-zhao changed the title Adding RESTCatalog based Spark Smoke Test Adding RESTCatalog based Spark Integ Test Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement PR that improves existing functionality
Projects
None yet
Development

No branches or pull requests

1 participant