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-36304][SQL] Refactor fifteenth set of 20 query execution errors to use error classes #34208

Closed
wants to merge 3 commits into from

Conversation

dchvn
Copy link
Contributor

@dchvn dchvn commented Oct 7, 2021

What changes were proposed in this pull request?

Adds error classes to some of the exceptions in QueryExecutionErrors.

Why are the changes needed?

Improves auditing for developers and adds useful fields for users (error class and SQLSTATE).

Does this PR introduce any user-facing change?

Fills in missing error class and SQLSTATE fields.

How was this patch tested?

Existing tests

@HyukjinKwon
Copy link
Member

cc @karenfeng FYI

@dchvn dchvn changed the title [SPARK-36304][SQL]: Refactor fifteenth set of 20 query execution errors to use error classes [SPARK-36304][SQL] Refactor fifteenth set of 20 query execution errors to use error classes Oct 7, 2021
@SparkQA
Copy link

SparkQA commented Oct 7, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48431/

@SparkQA
Copy link

SparkQA commented Oct 7, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48431/

@SparkQA
Copy link

SparkQA commented Oct 7, 2021

Test build #143908 has finished for PR 34208 at commit b228cae.

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

@SparkQA
Copy link

SparkQA commented Oct 7, 2021

Test build #143927 has finished for PR 34208 at commit b228cae.

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

@SparkQA
Copy link

SparkQA commented Oct 7, 2021

Test build #143956 has finished for PR 34208 at commit b228cae.

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

@SparkQA
Copy link

SparkQA commented Oct 7, 2021

Test build #143985 has finished for PR 34208 at commit b228cae.

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

@github-actions
Copy link

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!

@github-actions github-actions bot added the Stale label Jan 24, 2022
@dchvn dchvn closed this Jan 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants