Skip to content
This repository was archived by the owner on Jan 9, 2020. It is now read-only.

Conversation

@erikerlandson
Copy link
Member

We want to re-direct community dev upstream as much as possible. However, some contributions impact components (e.g. shuffle server) that do not yet exist upstream. To handle this, we decided to add this message and leave it up to developers, but encourage them to submit upstream unless it isn't feasible.

We want to re-direct community dev upstream as much as possible. However, some contributions impact components (e.g. shuffle server) that do not yet exist upstream. To handle this, we decided to add this message and leave it up to developers, but encourage them to submit upstream unless it isn't feasible.
Copy link
Member

@foxish foxish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

modulo one comment

Please consider whether your pull request can be submitted against the Apache Spark project, and submit there if possible.

If you have any questions about whether a PR should be submitted upstream or against this fork,
please feel free to ask on the Big Data SIG [slack channel](https://kubernetes.slack.com/),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's also add a link to the spark-dev mailing list here, and encourage use of that. Slack is good to keep around for questions about "where should this feature go?".

@foxish
Copy link
Member

foxish commented Feb 3, 2018

rerun integration test please

@erikerlandson erikerlandson merged commit a0117ea into branch-2.2-kubernetes Feb 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants