[SPARK-35021][SQL] Group exception messages in connector/catalog #32377
Conversation
|
Test build #138035 has finished for PR 32377 at commit
|
|
Kubernetes integration test unable to build dist. exiting with code: 1 |
|
Kubernetes integration test starting |
|
Kubernetes integration test status failure |
|
Test build #138037 has finished for PR 32377 at commit
|
|
retest this please |
|
Test build #138057 has finished for PR 32377 at commit
|
|
Kubernetes integration test unable to build dist. exiting with code: 1 |
|
Kubernetes integration test unable to build dist. exiting with code: 1 |
|
Test build #138065 has finished for PR 32377 at commit
|
|
ping @allisonwang-db |
| new AnalysisException(s"Cannot use catalog ${plugin.name}: $msg") | ||
| } | ||
|
|
||
| def invalidIdentifierAsItHasMoreThanTwoNamePartsError( |
There was a problem hiding this comment.
How about identifierHavingMoreThanTwoNamePartsError
| new AnalysisException(s"Cannot create tables with ${NullType.simpleString} type.") | ||
| } | ||
|
|
||
| def functionUnsupportedInV1CatalogError(): Throwable = { |
There was a problem hiding this comment.
functionUnsupportedInV2CatalogError
|
Kubernetes integration test starting |
|
Kubernetes integration test status failure |
|
Test build #138191 has finished for PR 32377 at commit
|
|
ping @allisonwang-db |
|
cc @cloud-fan |
|
@beliefer can you fix the conflicts? thanks! |
OK |
|
Kubernetes integration test unable to build dist. exiting with code: 1 |
|
thanks, merging to master! |
|
Test build #138245 has finished for PR 32377 at commit
|
|
@allisonwang-db Thank you for review. @cloud-fan Thank you too. |
What changes were proposed in this pull request?
This PR group exception messages in
sql/catalyst/src/main/scala/org/apache/spark/sql/connector/catalog.Why are the changes needed?
It will largely help with standardization of error messages and its maintenance.
Does this PR introduce any user-facing change?
No. Error messages remain unchanged.
How was this patch tested?
No new tests - pass all original tests to make sure it doesn't break any existing behavior.