Skip to content

[SQL][Minor] map -> foreach. #8862

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.

Sign up for GitHub

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

Jump to bottom
Closed
rxin wants to merge 1 commit into from
Closed

[SQL][Minor] map -> foreach. #8862

rxin wants to merge 1 commit into from

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Sep 22, 2015

DataFrame.explain should use foreach to print the explain content.

@rxin
[SQL][Minor] map -> foreach.
74eb746 
DataFrame.explain should use foreach to print the explain content.
@andrewor14
Copy link
Contributor

andrewor14 commented Sep 22, 2015

Merged into master 1.5.

asfgit pushed a commit that referenced this pull request Sep 22, 2015
@rxin
[SQL] [MINOR] map -> foreach.
a2b0fee 
DataFrame.explain should use foreach to print the explain content.

Author: Reynold Xin <[email protected]>

Closes #8862 from rxin/map-foreach.

(cherry picked from commit f3b727c)
Signed-off-by: Andrew Or <[email protected]>
@asfgit asfgit closed this in f3b727c Sep 22, 2015
@SparkQA
Copy link

SparkQA commented Sep 22, 2015

Test build #1784 has finished for PR 8862 at commit 74eb746.

  • This patch passes all tests.
  • This patch does not merge cleanly.
  • This patch adds the following public classes (experimental):
    • class Interaction(override val uid: String) extends Transformer
    • class AFTSurvivalRegression @Since("1.6.0") (@Since("1.6.0") override val uid: String)
    • require(censor == 1.0 || censor == 0.0, "censor of class AFTPoint must be 1.0 or 0.0")
    • class CountVectorizer(JavaEstimator, HasInputCol, HasOutputCol):
    • class CountVectorizerModel(JavaModel):
    • s"Failed to convert value $v (class of $
    • s"Failed to convert value $v (class of $
    • case class Sort(

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rxin @andrewor14 @SparkQA