Skip to content

[SPARK-16082][SparkR]remove duplicated docs in dapply#13790

Closed
NarineK wants to merge 1 commit intoapache:masterfrom
NarineK:dapply-docs-fix
Closed

[SPARK-16082][SparkR]remove duplicated docs in dapply#13790
NarineK wants to merge 1 commit intoapache:masterfrom
NarineK:dapply-docs-fix

Conversation

@NarineK
Copy link
Contributor

@NarineK NarineK commented Jun 20, 2016

What changes were proposed in this pull request?

Removed unnecessary duplicated documentation in dapply and dapplyCollect.

In this pull request I created separate R docs for dapply and dapplyCollect - kept dapply's documentation separate from dapplyCollect's and referred from one to another via a link.

How was this patch tested?

Existing test cases.

@NarineK
Copy link
Contributor Author

NarineK commented Jun 20, 2016

cc: @sun-rui, @shivaram @felixcheung

@SparkQA
Copy link

SparkQA commented Jun 20, 2016

Test build #60882 has finished for PR 13790 at commit 1331a09.

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

@shivaram
Copy link
Contributor

LGTM -- @felixcheung @sun-rui let me know if you have any comments

@felixcheung
Copy link
Member

LGTM

1 similar comment
@sun-rui
Copy link
Contributor

sun-rui commented Jun 21, 2016

LGTM

asfgit pushed a commit that referenced this pull request Jun 21, 2016
## What changes were proposed in this pull request?
Removed unnecessary duplicated documentation in dapply and dapplyCollect.

In this pull request I created separate R docs for dapply and dapplyCollect - kept dapply's documentation separate from dapplyCollect's and referred from one to another via a link.

## How was this patch tested?
Existing test cases.

Author: Narine Kokhlikyan <narine@slice.com>

Closes #13790 from NarineK/dapply-docs-fix.

(cherry picked from commit e2b7eba)
Signed-off-by: Shivaram Venkataraman <shivaram@cs.berkeley.edu>
@asfgit asfgit closed this in e2b7eba Jun 21, 2016
@NarineK NarineK changed the title remove duplicated docs in dapply [SPARK-16082][SparkR]remove duplicated docs in dapply Jun 21, 2016
@NarineK
Copy link
Contributor Author

NarineK commented Jun 21, 2016

@shivaram , I've noticed that I didn't associate the pull request with the jira. I've just did it.

@shivaram
Copy link
Contributor

Thanks -- I should have noticed it before the merge, but missed it. I have resolved the JIRA and put in a link to the PR there, so I think its all fine.

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

Comments