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

[SPARK-8355] [SQL] Python DataFrameReader/Writer should mirror Scala #7078

Closed
wants to merge 2 commits into from

Conversation

piaozhexiu
Copy link

I compared PySpark DataFrameReader/Writer against Scala ones. Option function is missing in both reader and writer, but the rest seems to all match.

I added Option to reader and writer and updated the pyspark-sql test.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@rxin
Copy link
Contributor

rxin commented Jun 29, 2015

Jenkins, add to whitelist.

@rxin
Copy link
Contributor

rxin commented Jun 29, 2015

Jenkins, ok to test.

@@ -74,6 +74,13 @@ def schema(self, schema):
return self

@since(1.4)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be 1.5

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for reviewing. Fixed.

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@SparkQA
Copy link

SparkQA commented Jun 29, 2015

Test build #35966 has started for PR 7078 at commit 524e0aa.

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@SparkQA
Copy link

SparkQA commented Jun 29, 2015

Test build #35968 has started for PR 7078 at commit c63d419.

@rxin
Copy link
Contributor

rxin commented Jun 29, 2015

lgtm.

@SparkQA
Copy link

SparkQA commented Jun 29, 2015

Test build #35966 has finished for PR 7078 at commit 524e0aa.

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

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@rxin
Copy link
Contributor

rxin commented Jun 29, 2015

@JoshRosen maybe a bug in the reporting script? it fails because it cannot find unit-tests.log

@rxin
Copy link
Contributor

rxin commented Jun 29, 2015

Going to merge this. Thanks!

@asfgit asfgit closed this in ac2e17b Jun 29, 2015
@SparkQA
Copy link

SparkQA commented Jun 29, 2015

Test build #35968 has finished for PR 7078 at commit c63d419.

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

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

asfgit pushed a commit that referenced this pull request Jun 29, 2015
I compared PySpark DataFrameReader/Writer against Scala ones. `Option` function is missing in both reader and writer, but the rest seems to all match.

I added `Option` to reader and writer and updated the `pyspark-sql` test.

Author: Cheolsoo Park <cheolsoop@netflix.com>

Closes #7078 from piaozhexiu/SPARK-8355 and squashes the following commits:

c63d419 [Cheolsoo Park] Fix version
524e0aa [Cheolsoo Park] Add option function to df reader and writer

(cherry picked from commit ac2e17b)
Signed-off-by: Reynold Xin <rxin@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants