Skip to content

[SPARK-16693][SPARKR] Remove methods deprecated in 2.0.0 or before - #14330

Closed
felixcheung wants to merge 4 commits into
apache:masterfrom
felixcheung:rremovedeprecate
Closed

[SPARK-16693][SPARKR] Remove methods deprecated in 2.0.0 or before#14330
felixcheung wants to merge 4 commits into
apache:masterfrom
felixcheung:rremovedeprecate

Conversation

@felixcheung

@felixcheungfelixcheung commented Jul 23, 2016

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

Remove deprecated functions which includes:
SQLContext/HiveContext stuff
sparkR.init
jsonFile
parquetFile
registerTempTable

How was this patch tested?

unit tests
@shivaram

@SparkQA

Copy link
Copy Markdown

Test build #62758 has finished for PR 14330 at commit 3162215.

  • This patch fails some tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@markhamstra

Copy link
Copy Markdown
Contributor

See JIRA comment.

@SparkQA

Copy link
Copy Markdown

Test build #62759 has finished for PR 14330 at commit 9cc2260.

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

@shivaram

Copy link
Copy Markdown
Contributor

Yeah lets discuss this on JIRA / mailing lists and then get back to this

@SparkQA

Copy link
Copy Markdown

Test build #72511 has finished for PR 14330 at commit 9cc2260.

  • This patch passes all tests.
  • This patch does not merge cleanly.
  • This patch adds no public classes.

@gatorsmile

Copy link
Copy Markdown
Member

Any update on this PR?

@HyukjinKwonHyukjinKwon mentioned this pull request Jun 25, 2017
@gatorsmile

Copy link
Copy Markdown
Member

We are closing it due to inactivity. please do reopen if you want to push it forward. Thanks!

zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 22, 2025
## What changes were proposed in this pull request?
This PR proposes to close stale PRs, mostly the same instances with apache#18017
I believe the author in apache#14807 removed his account.
Closesapache#7075Closesapache#8927Closesapache#9202Closesapache#9366Closesapache#10861Closesapache#11420Closesapache#12356Closesapache#13028Closesapache#13506Closesapache#14191Closesapache#14198Closesapache#14330Closesapache#14807Closesapache#15839Closesapache#16225Closesapache#16685Closesapache#16692Closesapache#16995Closesapache#17181Closesapache#17211Closesapache#17235Closesapache#17237Closesapache#17248Closesapache#17341Closesapache#17708Closesapache#17716Closesapache#17721Closesapache#17937
Added:
Closesapache#14739Closesapache#17139Closesapache#17445Closesapache#18042Closesapache#18359
Added:
Closesapache#16450Closesapache#16525Closesapache#17738
Added:
Closesapache#16458Closesapache#16508Closesapache#17714
Added:
Closesapache#17830Closesapache#14742
## How was this patch tested?
N/A
Author: hyukjinkwon <gurwls223@gmail.com>
Closesapache#18417 from HyukjinKwon/close-stale-pr.
Sign up for freeto 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

@felixcheung@SparkQA@markhamstra@shivaram@gatorsmile