* Added option ydb.WithSessionPoolSessionIdleTimeToLive for restric…

[asmyasnikov]

…t idle time of query sessions

• Fixed bug with leak of query transactions

Pull request type

Please check the type of change your PR introduces:

• Bugfix
• Feature
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no api changes)
• Build related changes
• Documentation content changes
• Other (please describe):

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Other information

[github-actions]

github.com/ydb-platform/ydb-go-sdk/v3

compatible changes

WithSessionPoolSessionIdleTimeToLive: added

summary

Base version: v3.80.3-0.20240913152610-200688bf7691 (master)
Cannot suggest a release version.
Can only suggest a release version when compared against the most recent version of this major: v3.80.2.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 80.15873% with 25 lines in your changes missing coverage. Please review.

Project coverage is 67.92%. Comparing base (200688b) to head (d26f929).

Files with missing lines	Patch %	Lines
internal/query/transaction.go	79.16%	10 Missing ⚠️
internal/query/config/options.go	0.00%	5 Missing ⚠️
options.go	0.00%	4 Missing ⚠️
internal/query/session/status.go	0.00%	2 Missing ⚠️
internal/query/result.go	91.66%	1 Missing ⚠️
internal/query/session.go	75.00%	1 Missing ⚠️
internal/query/session/session.go	93.75%	1 Missing ⚠️
internal/tx/id.go	87.50%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1458      +/-   ##
==========================================
+ Coverage   67.83%   67.92%   +0.08%     
==========================================
  Files         350      350              
  Lines       36288    36284       -4     
==========================================
+ Hits        24616    24645      +29     
+ Misses      10588    10560      -28     
+ Partials     1084     1079       -5     

Flag	Coverage Δ
experiment	72.11% <80.86%> (-0.07%)	⬇️
go-1.21.x	67.95% <78.04%> (-0.11%)	⬇️
go-1.22.x	65.67% <76.98%> (-0.13%)	⬇️
go-1.23.x	67.83% <80.15%> (+0.04%)	⬆️
integration	53.95% <67.82%> (+0.38%)	⬆️
macOS	40.99% <66.95%> (+0.06%)	⬆️
ubuntu	72.34% <80.86%> (+0.08%)	⬆️
unit	41.42% <67.46%> (+0.07%)	⬆️
windows	41.41% <67.46%> (+0.06%)	⬆️
ydb-23.3	47.54% <4.34%> (-0.21%)	⬇️
ydb-24.1	53.83% <67.82%> (+0.27%)	⬆️
ydb-nightly	72.11% <80.86%> (-0.07%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

🌋 Here are results of SLO test for Go SDK xorm:

Grafana Dashboard

[github-actions]

🌋 Here are results of SLO test for Go SDK database/sql:

Grafana Dashboard

[github-actions]

🌋 Here are results of SLO test for Native ydb-go-sdk/v3 over table-service:

Grafana Dashboard

[github-actions]

🌋 Here are results of SLO test for Native ydb-go-sdk/v3 over query-service:

Grafana Dashboard

[github-actions]

🌋 Here are results of SLO test for Go SDK gorm:

Grafana Dashboard