Skip to content

[SPARK-42838][SQL] Assign a name to the error class _LEGACY_ERROR_TEMP_2000 #48332

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

Closed
mihailomilosevic2001 wants to merge 22 commits into from
Closed

[SPARK-42838][SQL] Assign a name to the error class _LEGACY_ERROR_TEMP_2000 #48332

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
3a1bac1
Fix OracleIntegrationSuite for JDK8
mihailomilosevic2001 May 30, 2024
3d34080
Revert "Fix OracleIntegrationSuite for JDK8"
mihailomilosevic2001 May 30, 2024
f8470aa
Rename legacy error
mihailomilosevic2001 Oct 3, 2024
1e275ee
Fix , deletion
mihailomilosevic2001 Oct 3, 2024
13ee2b0
Fix ordering
mihailomilosevic2001 Oct 3, 2024
b2eec80
Add new error condition
mihailomilosevic2001 Oct 3, 2024
a2b9371
Improve error name
mihailomilosevic2001 Oct 3, 2024
468df0d
Update common/utils/src/main/resources/error/error-conditions.json
mihailomilosevic2001 Oct 3, 2024
a8efe0f
Update error-conditions.json
mihailomilosevic2001 Oct 3, 2024
84da3d3
Fix tests
mihailomilosevic2001 Oct 4, 2024
58336d2
Fix test
mihailomilosevic2001 Oct 4, 2024
cdd7e42
Incorporate changes
mihailomilosevic2001 Oct 4, 2024
6c3ee3d
Fix duplicates
mihailomilosevic2001 Oct 4, 2024
093cfe4
Fix codegen
mihailomilosevic2001 Oct 4, 2024
77dce65
Merge remote-tracking branch 'upstream/master' into invalid_date_argu…
mihailomilosevic2001 Nov 8, 2024
0843220
Fix error message and align with other PR
mihailomilosevic2001 Nov 8, 2024
42f3539
Fix problems
mihailomilosevic2001 Nov 8, 2024
03ef738
Fix wrong message
mihailomilosevic2001 Nov 12, 2024
d7c176a
Fix style
mihailomilosevic2001 Nov 12, 2024
fb34be0
Incorporate changes
mihailomilosevic2001 Nov 13, 2024
2a1f41c
Fix codeGen
mihailomilosevic2001 Nov 13, 2024
95643bb
Merge branch 'apache:master' into invalid_date_argument_value
mihailomilosevic2001 Nov 13, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix problems
  • Loading branch information
@mihailomilosevic2001
mihailomilosevic2001 committed Nov 8, 2024
commit 42f353941fa8ae623d88b8b44b094b4896633e14
2 changes: 1 addition & 1 deletion sql/catalyst/src/main/scala/org/apache/spark/sql/errors/QueryExecutionErrors.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@ private[sql] object QueryExecutionErrors extends QueryErrorsBase with ExecutionE

def ansiDateTimeArgumentOutOfRange(e: Exception): SparkDateTimeException = {
Copy link
Member

@MaxGekk MaxGekk Oct 6, 2024

Choose a reason for hiding this comment

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

Changes in this PR clash with #48242 by @itholic

Copy link
Contributor Author

@mihailomilosevic2001 mihailomilosevic2001 Oct 6, 2024

Choose a reason for hiding this comment

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

Let's resolve that ticket first, but keep this one open for now, as I am not sure what complete scope will be of the other ticket and this PRs goal is to remove _LEGACY_ERROR_TEMP_2000 completely.

Copy link
Contributor Author

@mihailomilosevic2001 mihailomilosevic2001 Nov 8, 2024

Choose a reason for hiding this comment

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

@MaxGekk I aligned my changes with the @itholic PR, so we can go on and merge this change, then the other PR can improve the message, while this one stays scoped to assigning proper error classes.

Copy link
Member

@MaxGekk MaxGekk Nov 9, 2024

Choose a reason for hiding this comment

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

sgtm

new SparkDateTimeException(
errorClass = "DATETIME_ARGUMENT_OUT_OF_RANGE",
errorClass = "DATETIME_FIELD_OUT_OF_BOUNDS",
messageParameters = Map(
"rangeMessage" -> e.getMessage,
"ansiConfig" -> toSQLConf(SQLConf.ANSI_ENABLED.key)),
Expand Down