feat(数据源): 数据源打包剔除JDBC驱动，以方便使用自己的JDBC驱动版本 #72
Conversation
|
Ice2Faith 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. |
数据源插件优化：JDBC驱动解耦与打包机制改进变更概述
变更文件
💡 小贴士与 lingma-agents 交流的方式📜 直接回复评论
📜 在代码行处标记
📜 在讨论中提问
|
![lingma-agents[bot]](https://avatars.githubusercontent.com/in/1198481?s=60&v=4){kind=small}
There was a problem hiding this comment.
🔎 代码评审报告
🎯 评审意见概览
| 严重度 | 数量 | 说明 |
|---|---|---|
| 🔴 Blocker | 0 | 阻断性问题，需立即修复。例如：系统崩溃、关键功能不可用或严重安全漏洞。 |
| 🟠 Critical | 1 | 严重问题，高优先级修复。例如：核心功能异常或性能瓶颈影响用户体验。 |
| 🟡 Major | 1 | 主要问题，建议修复。例如：非核心功能缺陷或代码维护性较差。 |
| 🟢 Minor | 2 | 次要问题，酬情优化。例如：代码格式不规范或注释缺失。 |
总计: 4 个问题
📋 评审意见详情
💡 代码实现建议
以下是文件级别的代码建议，聚焦于代码的可读性、可维护性和潜在问题。
🪟 nacos-datasource-plugin-ext/bash/collect.bat (2 💬)
- 批处理脚本在移动目录前未检查目标目录是否存在。 (L16)
- 批处理脚本中存在冗余的目录删除操作。 (L18)
☕ nacos-datasource-plugin-ext/nacos-kingbase-datasource-plugin-ext/src/main/java/com/alibaba/nacos/plugin/datasource/dialect/KingbaseDatabaseDialect.java (1 💬)
☕ nacos-datasource-plugin-ext/nacos-opengauss-datasource-plugin-ext/src/main/java/com/alibaba/nacos/plugin/datasource/dialect/GaussdbDatabaseDialect.java (1 💬)
- 类注释描述不准确，应更新为反映实际数据库类型。 (L20)
🚀 架构设计建议
以下是对代码架构和设计的综合分析，聚焦于跨文件交互、系统一致性和潜在优化空间。
🔍1. JDBC驱动解耦策略导致的潜在依赖管理复杂性增加
通过将JDBC驱动设置为provided和optional，虽然实现了驱动版本的解耦，允许用户自定义驱动版本，但这增加了用户在部署时的复杂性。用户需要手动确保JDBC驱动与数据源插件的兼容性，否则可能导致运行时错误。此外，这种策略在多数据源场景下可能引发版本冲突问题。建议在README中提供更详细的兼容性检查指南，并考虑提供自动化工具或脚本来验证驱动与插件的兼容性。
📌 关键代码
<scope>provided</scope>
<optional>true</optional>- 这样做的目的是为了将 JDBC 驱动进行解耦，以便于你可以选择自己更合适的 JDBC 驱动版本
- 而不是插件里面直接合并的驱动版本用户可能因驱动版本不兼容导致系统运行异常，增加部署和维护成本。
🔍2. 批处理脚本中的健壮性和效率问题
collect.bat脚本在执行目录操作时缺乏必要的错误检查和处理机制，如目标目录存在性检查和删除操作的原子性保证。此外，脚本中存在冗余的目录删除操作，可能影响执行效率。建议优化脚本逻辑，增加错误处理机制，并移除冗余操作以提高脚本的健壮性和执行效率。
📌 关键代码
mkdir %dst_path%
mkdir .\backup
rd /q /s .\backup
move /Y %dst_path% .\backup
rd /q /s %dst_path%
mkdir %dst_path%
脚本执行失败可能导致数据丢失或部署中断，影响系统的稳定性和可用性。
🔍3. 数据库方言实现中的代码复用问题
KingbaseDatabaseDialect直接复用了MySQL的函数枚举类TrustedMysqlFunctionEnum，这可能导致数据库方言实现的不准确性和潜在的兼容性问题。建议为Kingbase数据库创建独立的函数枚举类，以确保方言实现的准确性和可维护性。
📌 关键代码
import com.alibaba.nacos.plugin.datasource.enums.mysql.TrustedMysqlFunctionEnum;return TrustedMysqlFunctionEnum.getFunctionByName(functionName);数据库方言实现不准确可能导致SQL执行错误，影响数据源插件的稳定性和数据一致性。
🔍4. Maven插件配置的重复性和维护性问题
在多个数据源插件的pom.xml文件中重复配置了maven-dependency-plugin，这增加了配置的重复性和维护难度。建议将插件配置提取到父POM中，通过pluginManagement进行统一管理，以减少重复配置并提高维护效率。
📌 关键代码
<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-dependency-plugin</artifactId>
</plugin>
</plugins>
</build>配置重复可能导致维护困难和配置不一致，增加项目管理的复杂性。
审查详情
📒 文件清单 （12 个文件）
✅ 新增: 2 个文件
📝 变更: 10 个文件
✅ 新增文件:
nacos-datasource-plugin-ext/bash/.gitignorenacos-datasource-plugin-ext/bash/collect.bat
📝 变更文件:
nacos-datasource-plugin-ext/README.mdnacos-datasource-plugin-ext/nacos-dm-datasource-plugin-ext/pom.xmlnacos-datasource-plugin-ext/nacos-kingbase-datasource-plugin-ext/pom.xmlnacos-datasource-plugin-ext/nacos-kingbase-datasource-plugin-ext/src/main/java/com/alibaba/nacos/plugin/datasource/dialect/KingbaseDatabaseDialect.javanacos-datasource-plugin-ext/nacos-mssql-datasource-plugin-ext/pom.xmlnacos-datasource-plugin-ext/nacos-opengauss-datasource-plugin-ext/pom.xmlnacos-datasource-plugin-ext/nacos-opengauss-datasource-plugin-ext/src/main/java/com/alibaba/nacos/plugin/datasource/dialect/GaussdbDatabaseDialect.javanacos-datasource-plugin-ext/nacos-oracle-datasource-plugin-ext/pom.xmlnacos-datasource-plugin-ext/nacos-postgresql-datasource-plugin-ext/pom.xmlpom.xml
💡 小贴士
与 lingma-agents 交流的方式
📜 直接回复评论
直接回复本条评论，lingma-agents 将自动处理您的请求。例如：
-
在当前代码中添加详细的注释说明。
-
请详细介绍一下你说的 LRU 改造方案，并使用伪代码加以说明。
📜 在代码行处标记
在文件的特定位置创建评论并 @lingma-agents。例如：
-
@lingma-agents 分析这个方法的性能瓶颈并提供优化建议。
-
@lingma-agents 对这个方法生成优化代码。
📜 在讨论中提问
在任何讨论中 @lingma-agents 来获取帮助。例如：
-
@lingma-agents 请总结上述讨论并提出解决方案。
-
@lingma-agents 请根据讨论内容生成优化代码。
| mkdir .\backup | ||
|
|
||
| rd /q /s .\backup | ||
| move /Y %dst_path% .\backup |
There was a problem hiding this comment.
| rd /q /s .\backup | ||
| move /Y %dst_path% .\backup | ||
|
|
||
| rd /q /s %dst_path% |
|
|
||
| @Override | ||
| public String getFunction(String functionName) { | ||
| return TrustedMysqlFunctionEnum.getFunctionByName(functionName); |
There was a problem hiding this comment.
应使用与Kingbase数据库相关的函数枚举类，而不是复用MySQL的函数枚举。
🟠 Critical | 🐞 Bugs
📋 问题详情
在getFunction方法中直接调用了TrustedMysqlFunctionEnum.getFunctionByName(functionName)，这是不合适的，因为Kingbase数据库可能不支持所有MySQL函数，或者有自己特定的函数实现。应该使用Kingbase专用的函数枚举类来提供正确的函数映射。
💡 解决方案
修改getFunction方法中的函数获取逻辑，使其使用Kingbase专用的函数枚举类。
- return TrustedMysqlFunctionEnum.getFunctionByName(functionName);
+ return KingbaseFunctionEnum.getFunctionByName(functionName);您的反馈对我们很重要！(建议右键在新标签页中打开以下链接)
| package com.alibaba.nacos.plugin.datasource.dialect; | ||
|
|
||
| import com.alibaba.nacos.plugin.datasource.constants.DatabaseTypeConstant; | ||
| import com.alibaba.nacos.plugin.datasource.impl.enums.GaussdbFunctionEnum; |
There was a problem hiding this comment.
2 participants
从打包结果中剔除JDBC驱动，同时打包为release产物，支持直接下载
更新插件使用说明，即直接下载release产物放到nacos目录的plugins目录下即可
不再需要拉取源代码编译，也可以自己直接更换或者升级JDBC驱动版本，而不必源码更新JDBC驱动版本再打包