Add Sentinel MyBatis adapter module
Resolves #1113
Why we need it?
Database needs sentinel, because:
In most cases, the bottleneck of the system is the database. Although there are web or rpc sentinel, but some unimportant requests occupy database resources.
How to verify it?
Implements org.apache.ibatis.plugin.Interceptor
Junit:
module: sentinel-adapter/sentinel-mybatis-adapter com.alibaba.csp.sentinel.adapter.mybatis.test.TestMapperInterceptor com.alibaba.csp.sentinel.adapter.mybatis.test.TestMybatisTransactional com.alibaba.csp.sentinel.adapter.mybatis.test.TestReadInterceptor com.alibaba.csp.sentinel.adapter.mybatis.test.TestTotalInterceptor com.alibaba.csp.sentinel.adapter.mybatis.test.TestWriteInterceptor
Codecov Report
Merging #1118 into master will increase coverage by
0.05%. The diff coverage is56.60%.
@@ Coverage Diff @@
## master #1118 +/- ##
============================================
+ Coverage 43.53% 43.59% +0.05%
- Complexity 1677 1695 +18
============================================
Files 364 371 +7
Lines 10492 10543 +51
Branches 1409 1414 +5
============================================
+ Hits 4568 4596 +28
- Misses 5365 5384 +19
- Partials 559 563 +4
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update be4d058...c3de882. Read the comment docs.
Could you please also sync your code with the master branch?
Could you please also sync your code with the master branch?
OK.
Could you please also sync your code with the master branch?
OK.
@kaizi2009 Hi, any progress?
Friendly ping :)
Could you please also sync your code with the master branch?
OK.
@kaizi2009 Hi, any progress?
Done. Sorry, I'm busy these days.
@sczyh30 Please review the code.
@sczyh30 Please review the code. ( ^___^ )
@sczyh30 Merging is blocked。What do I need to do next?
Still in progress?
This appears to be a practical adapter, but we're closing this PR due to prolonged inactivity. If you'd like to continue contributing, feel free to create a new PR based on the 1.8 branch and we'll prioritize processing it promptly.