sandip-db commented on code in PR #44685:
URL: https://github.com/apache/spark/pull/44685#discussion_r1449173137
##########
common/utils/src/main/resources/error/error-classes.json:
##########
@@ -2661,6 +2661,11 @@
],
"sqlState" : "42K0E"
},
+ "MULTIPLE_XML_DATA_SOURCE" : {
Review Comment:
@MaxGekk `FOUND_MULTIPLE_DATA_SOURCES` is currently being used for user
defined functions.
For csv and other built-in formats, spark implicitly chooses the built-in
data source. While we expect that most of the spark-xml users would prefer to
use the new built-in XML, a few of them may still want to continue using the
external library. So we wanted to be explicit when an external xml data source
is detected by throwing an error with a descriptive message that tells users to
either use the full classpath name or remove the external library from the
classpath. This message won't be applicable to the UDF scenario for which
`FOUND_MULTIPLE_DATA_SOURCES` is being used.
`_LEGACY_ERROR_TEMP_1141` and `FOUND_MULTIPLE_DATA_SOURCES` seems
duplicative though.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
