Support new error format in arrow/tarantool_test.go / TestInsert_invalid
#415
Comments
oleg-jukovec
added a commit
that referenced
this issue
Oct 23, 2024
The pull request changes [1] changes an error message on arrow decoding. The patch replaces a check of error message to a check of error code as it is a more stable check. However, the code is changed too. So as a temporary solution we need to work with both codes at the same time. 1. tarantool/tarantool#10665 Part of #415
oleg-jukovec
added a commit
that referenced
this issue
Oct 24, 2024
The pull request changes [1] changes an error message on arrow decoding. The patch replaces a check of error message to a check of error code as it is a more stable check. However, the code is changed too. So as a temporary solution we need to work with both codes at the same time. 1. tarantool/tarantool#10665 Part of #415
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
changes the format of the xrow decoding error. Previously the error message was "Failed to decode Arrow IPC data".
Now the top-level error message equals "Invalid MsgPack - packet body" with a chain of nested errors.
So I suggest:
go-tarantool/arrow/tarantool_test.go
Lines 38 to 45 in d4b5d04
Merge the patch mentioned above to master.
Enable the checks with updated error messages.
The text was updated successfully, but these errors were encountered: