Skip to content

Conversation

@koeninger
Copy link
Contributor

No description provided.

@SparkQA
Copy link

SparkQA commented Oct 7, 2015

Test build #43315 has finished for PR 9007 at commit 2aa200b.

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

@srowen
Copy link
Member

srowen commented Oct 7, 2015

Looks OK to me.

@srowen
Copy link
Member

srowen commented Oct 9, 2015

@tdas are you OK with this one?

@tdas
Copy link
Contributor

tdas commented Oct 9, 2015

there are a number of classes that this exposes. I have to spend some time to think whether it is worth exposing all of that.

@srowen
Copy link
Member

srowen commented Oct 29, 2015

@tdas what do you think? just trying to get some resolution on this and I know you wanted to think.

@srowen
Copy link
Member

srowen commented Nov 4, 2015

@tdas do you mind if I merge this?

@yhilem
Copy link

yhilem commented Dec 30, 2015

For which spark version the merge of this pull request is planned?

@koeninger
Copy link
Contributor Author

@tdas @srowen Can we either get this merged, or officially close off the possibility of merging it?

People are still raising specific issues on a fairly regular basis that would be solved in general by this PR (e.g. https://issues.apache.org/jira/browse/SPARK-13106), and it would be good to have clear expectations one way or the other.

@srowen
Copy link
Member

srowen commented Feb 1, 2016

@tdas I'd like to merge unless you object. I also find this a useful developer API.

@srowen
Copy link
Member

srowen commented Feb 7, 2016

Merged to master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants