-
Notifications
You must be signed in to change notification settings - Fork 199
Upgrade client/spark to use Spark 4.0.0-preview1 #728
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
Conversation
This reverts commit 9cd1804.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and are all the code changes needed for compilation with the new spark version?
Some codes changes are for compilation. |
Some tests just fail silently. No error no exception. But the results are wrong. e.g. this limit pushdown. |
* Fix Spark implicits issue with Delta OSS 4.0 * Revert "Fix Spark implicits issue with Delta OSS 4.0" This reverts commit 9cd1804. * upgrade spark to 4.0.0-preview1 * fix test * fix test * fix test * fix test * fix test --------- Co-authored-by: Sun Cao <[email protected]>
* Upgrade client/spark to use Spark 4.0.0-preview1 (#728) * Fix Spark implicits issue with Delta OSS 4.0 * Revert "Fix Spark implicits issue with Delta OSS 4.0" This reverts commit 9cd1804. * upgrade spark to 4.0.0-preview1 * fix test * fix test * fix test * fix test * fix test --------- Co-authored-by: Sun Cao <[email protected]> * Upgrade spark version to staging 4.0.0 for client/spark build (#729) * Setting version to 4.0.0 * fix compile * Revert "fix compile" This reverts commit daa57ce. * fix compile * fix compile * fix compile * fix compile * fix compile * fix limit pushdown * fix tests * fix tests * add case --------- Co-authored-by: Sun Cao <[email protected]> --------- Co-authored-by: Sun Cao <[email protected]>
For client and spark build:
Server build is still using spark 3 and java 8