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.

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

[SPARKR] [SPARK-10308] Add %in% to the exported namespace #8473

Closed
wants to merge 1 commit into from

Conversation

shivaram
Copy link
Contributor

I also checked all the other functions defined in column.R, functions.R and DataFrame.R and everything else looked fine.

cc @yu-iskw

@shivaram shivaram changed the title [SPARKR] [SPARK-10308] Add %in% to the exported NAMESPACE [SPARKR] [SPARK-10308] Add %in% to the exported namespace Aug 27, 2015
@yu-iskw
Copy link
Contributor

yu-iskw commented Aug 27, 2015

LGTM

@SparkQA
Copy link

SparkQA commented Aug 27, 2015

Test build #41659 has finished for PR 8473 at commit be062e5.

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

@shivaram
Copy link
Contributor Author

Alright I'm merging this.

asfgit pushed a commit that referenced this pull request Aug 27, 2015
I also checked all the other functions defined in column.R, functions.R and DataFrame.R and everything else looked fine.

cc yu-iskw

Author: Shivaram Venkataraman <shivaram@cs.berkeley.edu>

Closes #8473 from shivaram/in-namespace.

(cherry picked from commit ad7f0f1)
Signed-off-by: Shivaram Venkataraman <shivaram@cs.berkeley.edu>
@asfgit asfgit closed this in ad7f0f1 Aug 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants