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
[SPARK-15171][SQL] Remove the references to deprecated method dataset.registerTempTable #13098
Conversation
Test build #58556 has finished for PR 13098 at commit
|
Test build #58558 has finished for PR 13098 at commit
|
Test build #58559 has finished for PR 13098 at commit
|
Test build #58561 has finished for PR 13098 at commit
|
1aa763e
to
d1ba75b
Compare
Test build #58564 has finished for PR 13098 at commit
|
d1ba75b
to
304902c
Compare
Test build #58567 has finished for PR 13098 at commit
|
304902c
to
a7165f2
Compare
Test build #58571 has finished for PR 13098 at commit
|
@@ -703,8 +703,8 @@ val rowRDD = people.map(_.split(",")).map(p => Row(p(0), p(1).trim)) | |||
// Apply the schema to the RDD. | |||
val peopleDataFrame = sqlContext.createDataFrame(rowRDD, schema) | |||
|
|||
// Register the DataFrames as a table. | |||
peopleDataFrame.registerTempTable("people") | |||
// Creates a temporary view. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Creates a temporary view using the DataFrame
Please update the PR title and description since it also removes deprecated call from examples and docs. |
@@ -1975,9 +1978,9 @@ class SQLQuerySuite extends QueryTest with SharedSQLContext { | |||
nestedStructData.select($"record.r1.*"), | |||
Row(1, 1) :: Row(1, 2) :: Row(2, 1) :: Row(2, 2) :: Row(3, 1) :: Row(3, 2) :: Nil) | |||
|
|||
// Try with a registered table | |||
// Try with a temporary view | |||
withTempTable("nestedStructTable") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also rename this utility method?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can do it in follow-ups though.
LGTM except for some minor comments. |
Just merged this to master, but it conflicts with branch-2.0. Trying to resolve manually. |
Also merged to branch-2.0. |
….registerTempTable ## What changes were proposed in this pull request? Update the unit test code, examples, and documents to remove calls to deprecated method `dataset.registerTempTable`. ## How was this patch tested? This PR only changes the unit test code, examples, and comments. It should be safe. This is a follow up of PR #12945 which was merged. Author: Sean Zhong <seanzhong@databricks.com> Closes #13098 from clockfly/spark-15171-remove-deprecation. (cherry picked from commit 25b315e) Signed-off-by: Cheng Lian <lian@databricks.com>
Test build #58730 has finished for PR 13098 at commit
|
Test build #58731 has finished for PR 13098 at commit
|
What changes were proposed in this pull request?
Update the unit test code, examples, and documents to remove calls to deprecated method
dataset.registerTempTable
.How was this patch tested?
This PR only changes the unit test code, examples, and comments. It should be safe.
This is a follow up of PR #12945 which was merged.