Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed data race using log.WithNames #1203

Merged
merged 2 commits into from
Apr 18, 2024
Merged

Conversation

rekby
Copy link
Member

@rekby rekby commented Apr 18, 2024

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

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):

@rekby rekby self-assigned this Apr 18, 2024
@rekby rekby added the bug Something isn't working label Apr 18, 2024
@rekby rekby force-pushed the fix-race-on-with-names branch from 6f27ef8 to 668e3c4 Compare April 18, 2024 10:57
Copy link

summary

Base version: v3.65.1 (master)
Suggested version: v3.65.2

@rekby rekby force-pushed the fix-race-on-with-names branch from 668e3c4 to 028a822 Compare April 18, 2024 10:58
Copy link

github-actions bot commented Apr 18, 2024

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

Grafana Dashboard

SLO-xorm

Copy link

github-actions bot commented Apr 18, 2024

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

Grafana Dashboard

SLO-database-sql

Copy link

github-actions bot commented Apr 18, 2024

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

Grafana Dashboard

SLO-gorm

Copy link

github-actions bot commented Apr 18, 2024

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

Grafana Dashboard

SLO-native-table

@rekby rekby force-pushed the fix-race-on-with-names branch 2 times, most recently from 7daabc0 to 8085c49 Compare April 18, 2024 11:57
@rekby rekby force-pushed the fix-race-on-with-names branch from 8085c49 to 0567481 Compare April 18, 2024 11:58
@neyguvj neyguvj self-requested a review April 18, 2024 12:00
CHANGELOG.md Outdated Show resolved Hide resolved
@asmyasnikov asmyasnikov changed the title Fixed data race in logger ydb/log/WithNames Fixed data race using log.WithNames Apr 18, 2024
@asmyasnikov asmyasnikov merged commit c21837a into master Apr 18, 2024
35 checks passed
@asmyasnikov asmyasnikov deleted the fix-race-on-with-names branch April 18, 2024 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants