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

Add View Support to Spark #7938

Closed
nastra opened this issue Jun 29, 2023 · 7 comments
Closed

Add View Support to Spark #7938

nastra opened this issue Jun 29, 2023 · 7 comments
Assignees

Comments

@nastra
Copy link
Contributor

nastra commented Jun 29, 2023

Feature Request / Improvement

Once #7880 is in, work could be started on supporting Views in Spark

Query engine

Spark

@singhpk234
Copy link
Contributor

@nastra @amogh-jahagirdar any plans for this, i think spark view interfaces are in it's just that complete wiring in spark is still pending via [1] , do you plan to pickup implementing spark view interfaces in iceberg so that once the complete wiring of view support in spark is merged then would start supporting it or is it like we should wait for spark view support to be complete and then we will start implementing the spark view interfaces in iceberg-spark ?

cc @jzhuge

[1] https://github.com/apache/spark/pull/39796/files

@nastra
Copy link
Contributor Author

nastra commented Oct 24, 2023

@singhpk234 I was planning to pick up apache/spark#39796, but I don't know yet whether we'd want to integrate those temporarily into Iceberg until they make it into Spark

@jzhuge
Copy link
Member

jzhuge commented Dec 4, 2023

Rebased apache/spark#39796 to latest master branch

@jzhuge
Copy link
Member

jzhuge commented Dec 4, 2023

@jzhuge
Copy link
Member

jzhuge commented Dec 4, 2023

Spark umbrella JIRA: SPARK-31357

@jzhuge
Copy link
Member

jzhuge commented Dec 22, 2023

Rebased apache/spark#44197 with fixes for review comments and some unit tests

@rdblue
Copy link
Contributor

rdblue commented Feb 2, 2024

I think we can call this one done with all of the PRs from @nastra that we've merged lately. Thanks @jzhuge and @nastra for getting this ready!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

4 participants