Skip to content

Conversation

@ChoviWu
Copy link
Contributor

@ChoviWu ChoviWu commented Sep 2, 2022

postgresql and oracle updateInterceptor used common configuration

@yu199195
Copy link
Member

yu199195 commented Sep 2, 2022

hi,pls check ci

@ChoviWu
Copy link
Contributor Author

ChoviWu commented Sep 2, 2022

hi,pls check ci

ok, I'll resubmit

@codecov-commenter
Copy link

codecov-commenter commented Sep 2, 2022

Codecov Report

Merging #3908 (8c4fe8e) into master (dc5a85b) will decrease coverage by 0.01%.
The diff coverage is 42.85%.

@@             Coverage Diff              @@
##             master    #3908      +/-   ##
============================================
- Coverage     70.47%   70.45%   -0.02%     
+ Complexity     6777     6774       -3     
============================================
  Files           922      922              
  Lines         25367    25370       +3     
  Branches       2308     2309       +1     
============================================
- Hits          17877    17875       -2     
- Misses         6119     6120       +1     
- Partials       1371     1375       +4     
Impacted Files Coverage Δ
...java/org/apache/shenyu/common/utils/GsonUtils.java 65.64% <20.00%> (-1.66%) ⬇️
...a/org/apache/shenyu/admin/config/MapperConfig.java 100.00% <100.00%> (ø)
...nyu/admin/mybatis/common/SqlUpdateInterceptor.java 92.85% <100.00%> (ø)
...ging/rocketmq/client/RocketMQLogCollectClient.java 61.22% <0.00%> (-4.09%) ⬇️
...tencent/cls/client/TencentClsLogCollectClient.java 69.13% <0.00%> (-2.47%) ⬇️
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 62.76% <0.00%> (-1.07%) ⬇️
...che/shenyu/sync/data/http/HttpSyncDataService.java 89.79% <0.00%> (+4.08%) ⬆️

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

@ChoviWu ChoviWu closed this Sep 5, 2022
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.

3 participants