Skip to content

[SPARK-16029][SPARKR] SparkR add dropTempView and deprecate dropTempTable#13753

Closed
felixcheung wants to merge 2 commits intoapache:masterfrom
felixcheung:rdroptempview
Closed

[SPARK-16029][SPARKR] SparkR add dropTempView and deprecate dropTempTable#13753
felixcheung wants to merge 2 commits intoapache:masterfrom
felixcheung:rdroptempview

Conversation

@felixcheung
Copy link
Member

What changes were proposed in this pull request?

Add dropTempView and deprecate dropTempTable

How was this patch tested?

unit tests

@shivaram @liancheng

@liancheng
Copy link
Contributor

LGTM pending Jenkins. Thanks!

@SparkQA
Copy link

SparkQA commented Jun 18, 2016

Test build #60757 has finished for PR 13753 at commit 0b2fe6f.

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

@SparkQA
Copy link

SparkQA commented Jun 18, 2016

Test build #60762 has finished for PR 13753 at commit d70154c.

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

@shivaram
Copy link
Contributor

Thanks @felixcheung @liancheng - LGTM. Merging this to master, branch-2.0

Are there any other catalog functions that have changed in Spark 2.0 that we also expose in SparkR ? Or are createTempTable and dropTempTable the only ones ?

asfgit pushed a commit that referenced this pull request Jun 20, 2016
…able

## What changes were proposed in this pull request?

Add dropTempView and deprecate dropTempTable

## How was this patch tested?

unit tests

shivaram liancheng

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes #13753 from felixcheung/rdroptempview.

(cherry picked from commit 36e812d)
Signed-off-by: Shivaram Venkataraman <shivaram@cs.berkeley.edu>
@asfgit asfgit closed this in 36e812d Jun 20, 2016
@felixcheung
Copy link
Member Author

Only registerTempTable and dropTempTable have name changes.

@liancheng
Copy link
Contributor

@shivaram I think we've covered all of them.

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.

4 participants