ydb-go-sdk

ydb-go-sdk copied to clipboard

Check SLO with local k8s cluster

8

## Pull request type Please check the type of change your PR introduces: - [ ] Bugfix - [ ] Feature - [ ] Code style update (formatting, renaming) -...

asmyasnikov

Now the sdk create decoder for every message in a topic: https://github.com/ydb-platform/ydb-go-sdk/blob/904939fb726cb1d29b2cbf0bc1b31282ee6ac18b/internal/topic/topicreadercommon/decoders.go#L36-L36 It will be good to create pool of encoders with .Reset(io.Reader) method and reuse it. It is pair...

rekby

Now the sdk create encoder for every message in a topic: https://github.com/ydb-platform/ydb-go-sdk/blob/904939fb726cb1d29b2cbf0bc1b31282ee6ac18b/internal/topic/topicwriterinternal/encoders.go#L41-L41 It will be good to create pool of encoders with .Reset() method and reuse it.

rekby

dev: table.Session.Execute does X3 allocations for every string(or bytes) byte in result

3

Every byte loaded from YDB via query gets allocated in heap three time. For example if client reads from YDB 2 GiB/sec of bytes, the go runtime will have to...

zveznicht

# Bug Report **YDB GO SDK version:** github.com/ydb-platform/ydb-go-sdk/v3 v3.77.0 **Environment** macos, tests with datarace detector **Current behavior:** Got DataRace report: WARNING: DATA RACE Read at 0x00c0015a7ce0 by goroutine 8132: runtime.mapiterinit()...

MaxShoshin

For example `query.Sessions.Query` https://github.com/ydb-platform/ydb-go-sdk/blob/master/query/client.go#L28 uses "github.com/ydb-platform/ydb-go-sdk/v3/internal/query/options" which is not accessible by sdk users. It blocks migrating from table service to query.

zveznicht

# Feature Request **Describe the Feature Request** Hi! In https://github.com/ydb-platform/ydb-go-sdk/issues/1390 I wanted to descibe the problem with infinite discovery events. They litter our traces, and I suppose they slow down...

pelageech

Hi! I noticed that my logs are littered with such messages: ``` 2024-08-08 11:55:32.778 WARN 'ydb.driver.conn.stream.RecvMsg' => failed {"error":"EOF","latency":"31.691µs","version":"3.76.2"} 2024-08-08 11:55:32.779 WARN 'ydb.driver.conn.stream.RecvMsg' => failed {"error":"transport/Canceled (code = 1, source...

pelageech

# Bug Report **YDB GO SDK version:** 3.76.3 **Environment** macOS Sonoma 14.2.1 amd64 **Current behavior:** ``` 2024/08/07 21:10:52 next rowEOF at 'github.com/ydb-platform/ydb-go-sdk/v3/internal/query.(*materializedResultSet).NextRow(result_set.go:79)' ``` **Expected behavior:** No error and query plan....

pelageech

# Feature Request **Describe the Feature Request** At the moment, we can either show all spans connected with reading `*query.Result` or nothing at all. The traces are not satisfactory, too....

pelageech