apache / incubator-seata

:fire: Seata is an easy-to-use, high-performance, open source distributed transaction solution.
https://seata.apache.org/
Apache License 2.0
25.25k stars 8.77k forks source link

optimize: optimize database driver cannot found message #6800

Closed lyl2008dsg closed 1 month ago

lyl2008dsg commented 1 month ago

Ⅰ. Describe what this PR did

make exception message generic for all database drivers

Ⅱ. Does this pull request fix one issue?

fixes #6760

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

It is a supplement to this PR https://github.com/apache/incubator-seata/pull/6765

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 52.15%. Comparing base (b0c2bb2) to head (c023e08). Report is 1 commits behind head on 2.x.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/apache/incubator-seata/pull/6800/graphs/tree.svg?width=650&height=150&src=pr&token=tbmHt2ZfxO&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)](https://app.codecov.io/gh/apache/incubator-seata/pull/6800?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) ```diff @@ Coverage Diff @@ ## 2.x #6800 +/- ## ============================================ - Coverage 52.16% 52.15% -0.01% Complexity 6311 6311 ============================================ Files 1067 1067 Lines 37303 37303 Branches 4430 4432 +2 ============================================ - Hits 19458 19456 -2 + Misses 15912 15911 -1 - Partials 1933 1936 +3 ``` | [Files with missing lines](https://app.codecov.io/gh/apache/incubator-seata/pull/6800?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [...eata/core/store/db/AbstractDataSourceProvider.java](https://app.codecov.io/gh/apache/incubator-seata/pull/6800?src=pr&el=tree&filepath=core%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fseata%2Fcore%2Fstore%2Fdb%2FAbstractDataSourceProvider.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-Y29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2VhdGEvY29yZS9zdG9yZS9kYi9BYnN0cmFjdERhdGFTb3VyY2VQcm92aWRlci5qYXZh) | `0.00% <ø> (ø)` | | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/apache/incubator-seata/pull/6800/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)