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-41574][SQL] Update _LEGACY_ERROR_TEMP_2009
as INTERNAL_ERROR
.
#39389
Conversation
messageParameters = Map.empty) | ||
def dataTypeOperationUnsupportedError(): Throwable = { | ||
SparkException.internalError( | ||
s"""Operation dataType is not supported""") |
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.
s"""Operation dataType is not supported""") | |
"The operation `dataType` is not supported.") |
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.
Could you apply this, please.
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.
Oh... didn't recognize that it's overwritten. 🙏
Just applied the change.
…xecutionErrors.scala Co-authored-by: Maxim Gekk <max.gekk@gmail.com>
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.
Please, change PR's title and description to match your actual changes.
sql/catalyst/src/main/scala/org/apache/spark/sql/errors/QueryExecutionErrors.scala
Outdated
Show resolved
Hide resolved
_LEGACY_ERROR_TEMP_2009
as INTERNAL_ERROR
.
thanks, merging to master! |
What changes were proposed in this pull request?
This PR proposes to update
_LEGACY_ERROR_TEMP_2136
asINTERNAL_ERROR
.Why are the changes needed?
We should turn error class into INTERNAL_ERROR when it's not triggered by user space.
Does this PR introduce any user-facing change?
No
How was this patch tested?
./build/sbt "sql/testOnly org.apache.spark.sql.SQLQueryTestSuite*