Skip to content

[SQL] Initial support for reporting location of error in sql string #4587

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.

Sign up for GitHub

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

Jump to bottom
Closed
marmbrus wants to merge 5 commits into from
Closed

[SQL] Initial support for reporting location of error in sql string #4587

marmbrus wants to merge 5 commits into from

Conversation

@marmbrus
Copy link
Contributor

@marmbrus marmbrus commented Feb 13, 2015

No description provided.

@SparkQA
Copy link

SparkQA commented Feb 13, 2015

Test build #27431 has started for PR 4587 at commit d45ff60.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 13, 2015

Test build #27432 has started for PR 4587 at commit f3efa51.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 13, 2015

Test build #27431 has finished for PR 4587 at commit d45ff60.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class AnalysisException protected[sql] (
    • implicit class AnalysisErrorAt(t: TreeNode[_])
    • case class Origin(

@AmplabJenkins
Copy link

AmplabJenkins commented Feb 13, 2015

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

@SparkQA
Copy link

SparkQA commented Feb 13, 2015

Test build #27432 has finished for PR 4587 at commit f3efa51.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class AnalysisException protected[sql] (
    • implicit class AnalysisErrorAt(t: TreeNode[_])
    • case class Origin(

@AmplabJenkins
Copy link

AmplabJenkins commented Feb 13, 2015

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

@rxin
Copy link
Contributor

rxin commented Feb 13, 2015

This is really cool

@marmbrus
more errors
e155dce
@marmbrus
Merge remote-tracking branch 'marmbrus/position' into position
395c019 
Conflicts:
	sql/catalyst/src/main/scala/org/apache/spark/sql/AnalysisException.scala
@SparkQA
Copy link

SparkQA commented Feb 14, 2015

Test build #27493 has started for PR 4587 at commit 395c019.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 14, 2015

Test build #27493 has finished for PR 4587 at commit 395c019.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class AnalysisException protected[sql] (
    • implicit class AnalysisErrorAt(t: TreeNode[_])
    • case class Origin(

@AmplabJenkins
Copy link

AmplabJenkins commented Feb 14, 2015

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

@SparkQA
Copy link

SparkQA commented Feb 15, 2015

Test build #27500 has started for PR 4587 at commit 0810052.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 15, 2015

Test build #27500 has finished for PR 4587 at commit 0810052.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class AnalysisException protected[sql] (
    • implicit class AnalysisErrorAt(t: TreeNode[_])
    • case class Origin(

@AmplabJenkins
Copy link

AmplabJenkins commented Feb 15, 2015

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

asfgit pushed a commit that referenced this pull request Feb 16, 2015
@marmbrus
[SQL] Initial support for reporting location of error in sql string
63fa123 
Author: Michael Armbrust <michael@databricks.com>

Closes #4587 from marmbrus/position and squashes the following commits:

0810052 [Michael Armbrust] fix tests
395c019 [Michael Armbrust] Merge remote-tracking branch 'marmbrus/position' into position
e155dce [Michael Armbrust] more errors
f3efa51 [Michael Armbrust] Update AnalysisException.scala
d45ff60 [Michael Armbrust] [SQL] Initial support for reporting location of error in sql string

(cherry picked from commit 104b2c4)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@asfgit asfgit closed this in 104b2c4 Feb 16, 2015
@marmbrus marmbrus deleted the position branch February 17, 2015 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@marmbrus @SparkQA @AmplabJenkins @rxin