CurtHagenlocher commented on code in PR #3092:
URL: https://github.com/apache/arrow-adbc/pull/3092#discussion_r2183003139
##########
csharp/src/Drivers/Databricks/RetryHttpHandler.cs:
##########
@@ -118,7 +118,7 @@ protected override async Task<HttpResponseMessage>
SendAsync(
}
throw new DatabricksException(
- lastErrorMessage ?? "Service temporarily unavailable and retry
timeout exceeded",
+ "[SQLState: 08001]" + (lastErrorMessage ?? "Service
temporarily unavailable and retry timeout exceeded"),
Review Comment:
There's a separate field on `DatabricksException` for `SqlState`. It's not
in the constructor, but you can do
`throw new DatabricksException(...).SetSqlState("08001");`
I don't know if that would also make the message come out correctly where
you want it to, but having the SqlState as a property without having to parse
it out of the message is definitely a good thing.
--
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]
