You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "miland-db (via GitHub)" <gi...@apache.org> on 2026/03/03 15:43:30 UTC
[PR] [SPARK-55816][SQL] Rename _LEGACY_ERROR_TEMP_3058 to DUPLICATE_PARTITION_TRANSFORM with SQL state 42000 [spark]
miland-db opened a new pull request, #54596:
URL: https://github.com/apache/spark/pull/54596
### What changes were proposed in this pull request?
This PR adds a proper error class and SQL state for the `checkTransformDuplication` exception in `SchemaUtils`.
Changes:
- Added `DUPLICATE_PARTITION_TRANSFORM` error class to `error-conditions.json` with SQL state `42711`
- Updated `SchemaUtils.checkTransformDuplication()` to use the new error class
- Removed `_LEGACY_ERROR_TEMP_3058` legacy error class
### Why are the changes needed?
Previously, this error used `_LEGACY_ERROR_TEMP_3058` without a SQL state, making it harder
to identify and handle programmatically in error monitoring systems.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Updated `DataSourceV2SQLSuite` tests to check for new error class.
### Was this patch authored or co-authored using generative AI tooling?
No.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [SPARK-55816][SQL] Rename _LEGACY_ERROR_TEMP_3058 to DUPLICATE_PARTITION_TRANSFORM with SQL state 42711 [spark]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #54596:
URL: https://github.com/apache/spark/pull/54596#issuecomment-4696888925
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!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [SPARK-55816][SQL] Rename _LEGACY_ERROR_TEMP_3058 to DUPLICATE_PARTITION_TRANSFORM with SQL state 42711 [spark]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed pull request #54596: [SPARK-55816][SQL] Rename _LEGACY_ERROR_TEMP_3058 to DUPLICATE_PARTITION_TRANSFORM with SQL state 42711
URL: https://github.com/apache/spark/pull/54596
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org