Skip to content

Conversation

@marmbrus
Copy link
Contributor

We don't generally make things in catalyst/execution private. Instead they are just undocumented due to their lack of stability guarantees.

@marmbrus
Copy link
Contributor Author

/cc @rxin

@rxin
Copy link
Contributor

rxin commented Jul 26, 2016

LGTM

@SparkQA
Copy link

SparkQA commented Jul 26, 2016

Test build #62851 has finished for PR 14356 at commit e5dad1b.

  • This patch fails PySpark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@rxin
Copy link
Contributor

rxin commented Jul 26, 2016

Merging in master/2.0.

@asfgit asfgit closed this in f99e34e Jul 26, 2016
asfgit pushed a commit that referenced this pull request Jul 26, 2016
We don't generally make things in catalyst/execution private.  Instead they are just undocumented due to their lack of stability guarantees.

Author: Michael Armbrust <michael@databricks.com>

Closes #14356 from marmbrus/patch-1.

(cherry picked from commit f99e34e)
Signed-off-by: Reynold Xin <rxin@databricks.com>
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.

3 participants