ecodeclub / eorm

简单 ORM 框架
Apache License 2.0
194 stars 64 forks source link

feature: 分库分表:datasource-简单的分布式事务方案支持 #210

Closed Stone-afk closed 1 year ago

codecov[bot] commented 1 year ago

Codecov Report

Merging #210 (ce752f1) into dev (d918cee) will decrease coverage by 1.16%. The diff coverage is 66.51%.

:exclamation: Current head ce752f1 differs from pull request most recent head 078d2d1. Consider uploading reports for the commit 078d2d1 to get more accurate results

@@            Coverage Diff             @@
##              dev     #210      +/-   ##
==========================================
- Coverage   85.65%   84.49%   -1.16%     
==========================================
  Files          48       51       +3     
  Lines        3653     3858     +205     
==========================================
+ Hits         3129     3260     +131     
- Misses        415      483      +68     
- Partials      109      115       +6     
Impacted Files Coverage Δ
...l/datasource/shardingsource/sharding_datasource.go 56.52% <40.74%> (-20.41%) :arrow_down:
internal/datasource/cluster/cluster_db.go 60.00% <45.45%> (-21.82%) :arrow_down:
...al/datasource/transaction/bin_multi_transaction.go 48.00% <48.00%> (ø)
internal/datasource/transaction/types.go 91.66% <91.66%> (ø)
...ternal/datasource/transaction/delay_transaction.go 93.93% <93.93%> (ø)
sharding_select.go 73.83% <100.00%> (+0.18%) :arrow_up:

... and 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Stone-afk commented 1 year ago

mark

Stone-afk commented 1 year ago

暂时增加了一个 GetMultiV2

flycash commented 1 year ago

我合并进去然后我要重构了,等我先捋一下重构思路再说。当然,你也可以直接重构

Stone-afk commented 1 year ago

可以开个issue,讨论一下思路