Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

新增对halo数据库的支持 #5632

Closed
wants to merge 2 commits into from
Closed

Conversation

HaloLab001
Copy link

您好,halo是我司推出的一款关系型数据库,因使用druid出现无法找到halo这个数据库驱动,因此合并代码进行适配

@CLAassistant
Copy link

CLAassistant commented Dec 19, 2023

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ HaloLab001
❌ zhoulanqing


zhoulanqing seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@lizongbo
Copy link
Collaborator

哪里有这个数据库的详细介绍?driver class是可以自己配置,不一定需要加默认识别适配

@codecov-commenter
Copy link

Codecov Report

Attention: 1436 lines in your changes are missing coverage. Please review.

Comparison is base (1c89d31) 58.79% compared to head (ead80e0) 58.68%.
Report is 345 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #5632      +/-   ##
============================================
- Coverage     58.79%   58.68%   -0.11%     
- Complexity    27033    27698     +665     
============================================
  Files          1282     1331      +49     
  Lines        123622   127444    +3822     
  Branches      23288    23991     +703     
============================================
+ Hits          72680    74787    +2107     
- Misses        41854    43340    +1486     
- Partials       9088     9317     +229     
Files Coverage Δ
core/src/main/java/com/alibaba/druid/DbType.java 92.30% <100.00%> (+0.30%) ⬆️
core/src/main/java/com/alibaba/druid/VERSION.java 50.00% <ø> (ø)
...a/com/alibaba/druid/pool/DruidPooledStatement.java 68.28% <100.00%> (ø)
...om/alibaba/druid/pool/PreparedStatementHolder.java 90.62% <100.00%> (-9.38%) ⬇️
...ol/ha/selector/StickyRandomDataSourceSelector.java 0.00% <ø> (ø)
...druid/pool/vendor/MySqlValidConnectionChecker.java 76.19% <100.00%> (+25.31%) ⬆️
...a/druid/proxy/jdbc/CallableStatementProxyImpl.java 99.55% <100.00%> (+0.72%) ⬆️
...a/com/alibaba/druid/proxy/jdbc/NClobProxyImpl.java 100.00% <ø> (+20.00%) ⬆️
...a/druid/proxy/jdbc/PreparedStatementProxyImpl.java 91.07% <100.00%> (+2.66%) ⬆️
...m/alibaba/druid/proxy/jdbc/StatementProxyImpl.java 94.03% <100.00%> (-0.32%) ⬇️
... and 76 more

... and 177 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c493f8...ead80e0. Read the comment docs.

@lizongbo
Copy link
Collaborator

lizongbo commented May 4, 2024

代码存在冲突,请解决冲突后重新提交pr

@lizongbo lizongbo closed this May 4, 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.

None yet

4 participants