Exceptions thrown do not match documentation about rethinkdb.driver HOT 8 CLOSED

Originally, I kept them wrapped in AggregateException. But I had a change of heart ❤️ after thinking about it driving home. They are now unwrapped when used against the synchronous API and work as naturally expected in a synchronous world. Hehe.... catch that ? "a synchronous" 😎

from rethinkdb.driver.

Hey Oliver, how are you doing? Hope you're doing well. Sorry for the delayed response, still on vacation. Be back at work around April 25th or so. I'll try to answer as best I can. Thanks VERY much for the JSON traces -- they help a bunch.

First, it looks like you're using a Run Helper since the error message looks like:

The query response cannot be converted to an object of T or List<T>.
This run helper works with SUCCESS_ATOM results.....

Run Helpers aren't part of the official API, so I'm giving myself a little wiggle room here. 😉 😉 .

_I think_, what's happening here is { "t":18 is an error response from the server and the .RunHelper*<T> is expecting a non-error response so what you're getting instead is a ReqlDriverError because the .RunHelper can't magically convert an error response into a data response you request (possibly .RunResult).. IIRC, the Run Helpers don't do any checks for error responses instead they make a direct attempt to convert the JSON on the wire to the type you're expecting.

If you use the dynamic .Run method, you should get the expected exception you're looking for and error responses should line up with the Java driver.

Regardless, you bring up a good usability issue. We should probably check for error responses before attempting a direct Run Helper conversion.

Also, the core of the driver is async so the TPL/CLR will generate an AggregateException wrapping the underlying exception. I'm not sure what the official guidance is on unwrapping these exceptions for framework libraries. I don't have my reference books with me at the moment. I'll have to check when I get home.

Hope this helps,
Brian

from rethinkdb.driver.

Thanks for the response, yes I am using the non dynamic versions like RunResult or RunAtom. I prefer not to deal with dynamic unless absolutely necessary.

W.r.t AggregateException, I don't mind them if they're documented. I'm not sure if there's a standard practice for C# libraries, but dealing with actual exceptions is way more friendly.

from rethinkdb.driver.

Thanks again @oliverjanik. This is now fixed in master. Next release will contain the fix. We'll check and throw the response error for all Run Helpers before bottoming out with a ReqlDriverError in case of a mis-matched conversion.

from rethinkdb.driver.

Did you keep AggregateExceptions? Or do you unwrap them?

from rethinkdb.driver.

Any ETA on the release? Would make my life easier.

from rethinkdb.driver.

@oliverjanik v2.3.1-beta-1 released with changes thus far. Keyword: beta. There's some code reviewing in master that I still haven't finished. Also, have not tested new authentication in .NET Core yet.

from rethinkdb.driver.

OK, I'll give it a spin. Thank you very much.

from rethinkdb.driver.