Skip to content

Conversation

@jyotiska
Copy link
Contributor

SQL example code for Python, as shown on SQL Programming Guide

@SparkQA
Copy link

SparkQA commented Sep 24, 2014

QA tests have started for PR 2521 at commit 0b46148.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 24, 2014

QA tests have finished for PR 2521 at commit 0b46148.

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

@SparkQA
Copy link

SparkQA commented Sep 24, 2014

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20754/

@SparkQA
Copy link

SparkQA commented Sep 24, 2014

QA tests have started for PR 2521 at commit c90502a.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 24, 2014

QA tests have finished for PR 2521 at commit c90502a.

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

@SparkQA
Copy link

SparkQA commented Sep 24, 2014

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20756/

@SparkQA
Copy link

SparkQA commented Sep 24, 2014

QA tests have started for PR 2521 at commit 306667e.

  • This patch merges cleanly.

Copy link
Contributor

Choose a reason for hiding this comment

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

This assume that this script will be run at SPARK_HOME, it will be broken if user run it at SPARK_HOME/examples/src/main/python

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In that case, should the JSON file be supplied as sys.argv[1]?

Copy link
Contributor

Choose a reason for hiding this comment

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

How about

path = os.environ['SPARK_HOME'] + "examples/src/main/resources/people.json"

@davies
Copy link
Contributor

davies commented Sep 24, 2014

This example only demonstrate jsonFile(), it will more powerful if it could have some usage of inferSchema() and applySchema().

@SparkQA
Copy link

SparkQA commented Sep 24, 2014

QA tests have finished for PR 2521 at commit 306667e.

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

@SparkQA
Copy link

SparkQA commented Sep 24, 2014

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20757/

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have started for PR 2521 at commit 83d5ab7.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have finished for PR 2521 at commit 83d5ab7.

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

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20783/

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have started for PR 2521 at commit 43fd10a.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have finished for PR 2521 at commit 43fd10a.

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

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20785/

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20784/

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have started for PR 2521 at commit b25e436.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have finished for PR 2521 at commit b25e436.

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

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20796/

@jyotiska
Copy link
Contributor Author

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have started for PR 2521 at commit b25e436.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have finished for PR 2521 at commit b25e436.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20798/

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have started for PR 2521 at commit 1471dcb.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have finished for PR 2521 at commit 1471dcb.

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

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20801/

@jyotiska
Copy link
Contributor Author

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have started for PR 2521 at commit 1471dcb.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 25, 2014

QA tests have finished for PR 2521 at commit 1471dcb.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20806/

@jyotiska
Copy link
Contributor Author

@davies I have made the changes you asked for. Can you verify this?

@davies
Copy link
Contributor

davies commented Oct 1, 2014

@jyotiska Cool, look good to me, thanks!

@marmbrus
Copy link
Contributor

marmbrus commented Oct 1, 2014

Thanks! I've merged this to master.

@asfgit asfgit closed this in 17333c7 Oct 1, 2014
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.

5 participants