Skip to content

Conversation

@dingsongjie
Copy link
Contributor

@dingsongjie dingsongjie commented Jan 19, 2024

mongodb busiCall 中有些场景需要直接在内部终止事务,目前只能通过抛出异常,被MongoCall中的catch捕获后,终止。我添加了一个新的MongoCall方法,并且在busiCall Function中 添加了Task<bool>返回值

}
}

public static async Task MongoCall(this BranchBarrier bb, IMongoClient mc, Func<IClientSessionHandle, Task<bool>> busiCall)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

两个方法代码重合度很高,最好做一个抽取

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

两个方法代码重合度很高,最好做一个抽取

好的,我重新提交了

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@codecov
Copy link

codecov bot commented Jan 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c86f875) 70.99% compared to head (b109d13) 69.10%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #74      +/-   ##
==========================================
- Coverage   70.99%   69.10%   -1.90%     
==========================================
  Files          44       44              
  Lines        1903     1903              
  Branches        0      180     +180     
==========================================
- Hits         1351     1315      -36     
  Misses        552      552              
- Partials        0       36      +36     

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

@catcherwong catcherwong merged commit ed9067a into dtm-labs:main Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants