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.

Sign up for GitHub

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

Jump to bottom

fix: UpdateStmt doesn't update the statement correctly in sqlx/bulkinserter.go #3607

Merged
merged 3 commits into from
Oct 9, 2023
Merged

fix: UpdateStmt doesn't update the statement correctly in sqlx/bulkinserter.go #3607

merged 3 commits into from
Oct 9, 2023

Conversation

kevwan
Copy link
Contributor

@kevwan kevwan commented Oct 8, 2023

No description provided.

@codecov
Copy link

codecov bot commented Oct 8, 2023 •
edited
Loading

Codecov Report

Merging #3607 (93dc1f0) into master (5aedd9c) will increase coverage by 0.03%.
The diff coverage is 100.00%.

Files Coverage Δ
core/mapping/utils.go 100.00% <ø> (ø)
core/stores/sqlx/bulkinserter.go 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

@kevwan kevwan force-pushed the fix/bulkinserter-updatestmt branch from a9a2153 to 4a7384d Compare October 9, 2023 13:01
@kevwan kevwan force-pushed the fix/bulkinserter-updatestmt branch from 4a7384d to 93dc1f0 Compare October 9, 2023 13:47
@kevwan kevwan merged commit abd1fa9 into zeromicro:master Oct 9, 2023
@kevwan kevwan deleted the fix/bulkinserter-updatestmt branch October 9, 2023 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kevwan