Skip to content

[SPARKR][DOCS] update R API doc for subset/extract #16721

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.

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

Closed
wants to merge 3 commits into from

Conversation

felixcheung
Copy link
Member

@felixcheung felixcheung commented Jan 27, 2017

What changes were proposed in this pull request?

With extract [[ or replace [[<-, the parameter i is a column index, that needs to be corrected in doc. Also a few minor updates: examples, links.

How was this patch tested?

manual

@SparkQA
Copy link

SparkQA commented Jan 27, 2017

Test build #72089 has finished for PR 16721 at commit 2c1f673.

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

@SparkQA
Copy link

SparkQA commented Jan 27, 2017

Test build #72090 has finished for PR 16721 at commit de56852.

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

@felixcheung felixcheung reopened this Jan 28, 2017
@asfgit asfgit closed this in be7425e Jan 31, 2017
asfgit pushed a commit that referenced this pull request Jan 31, 2017
## What changes were proposed in this pull request?

backport #16721 to branch-2.1

## How was this patch tested?

manual

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes #16749 from felixcheung/rsubsetdocbackport.
cmonkey pushed a commit to cmonkey/spark that referenced this pull request Feb 15, 2017
## What changes were proposed in this pull request?

With extract `[[` or replace `[[<-`, the parameter `i` is a column index, that needs to be corrected in doc. Also a few minor updates: examples, links.

## How was this patch tested?

manual

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes apache#16721 from felixcheung/rsubsetdoc.
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.

2 participants