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

[SPARK-24516][K8S] Change Python default to Python3 #22810

Closed
wants to merge 1 commit into from

Conversation

ifilonenko
Copy link
Contributor

What changes were proposed in this pull request?

As this is targeted for 3.0.0 and Python2 will be deprecated by Jan 1st, 2020, I feel it is appropriate to change the default to Python3. Especially as these projects found here are deprecating their support.

How was this patch tested?

Unit and Integration tests

@ifilonenko
Copy link
Contributor Author

@felixcheung for merge and @kokes for opinion and raising this JIRA

@SparkQA
Copy link

SparkQA commented Oct 23, 2018

@SparkQA
Copy link

SparkQA commented Oct 23, 2018

Test build #97948 has finished for PR 22810 at commit ab03cc4.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Oct 23, 2018

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/4421/

@ifilonenko
Copy link
Contributor Author

or @holdenk for merge :)

@holdenk
Copy link
Contributor

holdenk commented Oct 25, 2018

Let me look on Friday during weekly review time.

@felixcheung
Copy link
Member

merged to master

@asfgit asfgit closed this in 19ada15 Oct 25, 2018
jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
## What changes were proposed in this pull request?

As this is targeted for 3.0.0 and Python2 will be deprecated by Jan 1st, 2020, I feel it is appropriate to change the default to Python3. Especially as these projects [found here](https://python3statement.org/) are deprecating their support.

## How was this patch tested?

Unit and Integration tests

Author: Ilan Filonenko <ifilondz@gmail.com>

Closes apache#22810 from ifilonenko/SPARK-24516.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants