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.

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

[SPARK-4932] Add help comments in Analytics #3775

Closed
wants to merge 1 commit into from
Closed

[SPARK-4932] Add help comments in Analytics #3775

wants to merge 1 commit into from

Conversation

maropu
Copy link
Member

@maropu maropu commented Dec 23, 2014

Trivial modifications for usability.

@maropu maropu changed the title Add help comments in Analytics [SPARK-4932] Add help comments in Analytics Dec 23, 2014
@JoshRosen
Copy link
Contributor

Jenkins, this is ok to test.

@JoshRosen
Copy link
Contributor

LGTM pending Jenkins.

@SparkQA
Copy link

SparkQA commented Dec 23, 2014

Test build #24744 has finished for PR 3775 at commit fbea8f5.

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

asfgit pushed a commit that referenced this pull request Dec 23, 2014
@maropu @JoshRosen
[SPARK-4932] Add help comments in Analytics
e74ce14 
Trivial modifications for usability.

Author: Takeshi Yamamuro <linguin.m.s@gmail.com>

Closes #3775 from maropu/AddHelpCommentInAnalytics and squashes the following commits:

fbea8f5 [Takeshi Yamamuro] Add help comments in Analytics

(cherry picked from commit 9c251c5)
Signed-off-by: Josh Rosen <joshrosen@databricks.com>
@JoshRosen
Copy link
Contributor

I've merged this into master (1.3.0) and branch-1.2 (1.2.1). Thanks!

@asfgit asfgit closed this in 9c251c5 Dec 23, 2014
@maropu maropu deleted the AddHelpCommentInAnalytics branch July 5, 2017 11:41
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
@maropu @JoshRosen @SparkQA