-
Notifications
You must be signed in to change notification settings - Fork 614
[LIVY-1019] Deprecate Spark 2 in livy #489
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
base: master
Are you sure you want to change the base?
Conversation
|
Thank you for your PR @ArnavBalyan! This could be an important step of cleaning up unneeded features in an upcoming release. The change is quite big, so it will take some time to review. In the meantime, can I ask why you chose Spark version 3.3.4 in the main |
Hi @gyogal thanks so much, really appreciate it. For Spark, looks like |
Oh right, it makes sense now, thanks! In #478 it has already been changed to version 3 in the |
Thanks @gyogal I can bump it up as a follow up change if it looks good to you? |
|
cc @gyogal gentle reminder thanks! 😀 |
|
@ArnavBalyan , I think both this and the Python 3 commit could be merged after |
|
Hi @gyogal thanks for taking a look! Makes perfect sense, will wait until we do the release. Let me know if you need support with release |
|
Hi @ArnavBalyan , the main Livy version has been updated to 0.10-incubating-SNAPSHOT, however this caused some conflicts in your PR. Could you please resolve them? Once that is done, I think this could be merged into version 0.10-incubating. |
What changes were proposed in this pull request?
How was this patch tested?