Oleg Jukovec
Oleg Jukovec
I don't have a stable [test](https://github.com/tarantool/go-tarantool/blob/0ffd926e219c543fa95f36c7aec25f729749e371/queue/example_connection_pool_test.go#L112-L222) results if `queue.cfg({in_replicaset = true})` is called after `box.cfg()`. I have such errors on re-identify after a role switch: ``` Unable to re-identify in...
Спасибо большое за образ, получилось немного его уменьшить (на amd64 с 845 до 521 МБ).
**Bug description** The server could not correctly map an index from a string passed with IPROTO_INDEX_NAME to a number. I found the problem with the `tarantool/go-tarantool` connector, but I'm able...
Now we do a lot of allocations per a request: ``` $ go test -bench BenchmarkClientSerialTyped -benchmem BenchmarkClientSerialTyped-12 12250 94545 ns/op 898 B/op 16 allocs/op ``` There is no need...
I made mistakes in some types. I used pointers receivers instead of immutable receivers. It leads to inefficient work to GC, see https://medium.com/a-journey-with-go/go-should-i-use-a-pointer-instead-of-a-copy-of-my-struct-44b43b104963 https://www.sobyte.net/post/2021-10/go-pointer/ But even in spite of this...
We need to review changes from go 1.13 to go 1.20 and update the code according to new features and recommendations.
The Tarantool 3.0 allows to enforce transaction to be synchronous. We could support the feature. See: 1. https://github.com/tarantool/tarantool/issues/8650 2. https://github.com/tarantool/tarantool/blob/03c184d4da9be9e1431762989833a6762a704cfb/src/box/iproto_constants.h#L218-L221
Now tuples are decoded into structures according to the order of the fields in the structure. We need to support tuple format in IPROTO features to decode tuples into structures...
At now `errcheck` is disabled for the `golangci-lint` check. https://github.com/tarantool/go-tarantool/blob/d8df65dcd0f29a6a5d07472115cbcf2753b12609/.golangci.yaml#L4-L6 We could enable it for the better linter coverage.
It will be easier for a user to read tuples in a human-readable format with `fmt.Println(tuple)`.