garlandz-db commented on PR #52589: URL: https://github.com/apache/spark/pull/52589#issuecomment-3790290752
New client Error class is null AND sql state is NULL Set error class to CONNECT_CLIENT_UNEXPECTED_MISSING_SQL_STATE ; sql state will be read from json file Error class is null AND sql state is not NULL Do nothing; Error class can be null since sql state is all we care about Error class is NOT null and sql state is NULL It is possible to read from the json file therefore error class should be used But if it cannot read then set sql state to XXKCM since we cant read it Error class is NOT null and sql state is NOT NULL Do nothing; this is already constructed Error is not even a SparkThrowable convert it into one and set error class to CONNECT_CLIENT_UNEXPECTED_MISSING_SQL_STATE ; Old client Error class is null AND sql state is NULL same Error class is null AND sql state is not NULL same Error class is NOT null and sql state is NULL same Error class is NOT null and sql state is NOT NULL Same Error is not even a SparkThrowable Do not touch -- 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]
