Skip to content

fix(mysql): add FROM and connector integration tests - #884

Closed
HarsenLin wants to merge 2 commits into
developfrom
fix/TAP-12729-mysql-batch-read-sql-v2
Closed

fix(mysql): add FROM and connector integration tests#884
HarsenLin wants to merge 2 commits into
developfrom
fix/TAP-12729-mysql-batch-read-sql-v2

Conversation

@HarsenLin

@HarsenLin HarsenLin commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

变更说明

修复 MySQL 连接器在字段数大于 50 时批量读取 SQL 缺少 FROM 的问题,并以安全 squash 方式合入 feature-it 的 Connector Integration Tests。

  • 修正大字段表查询为 SELECT * FROM <schema.table>
  • 增加 51 字段精确 SQL 回归测试。
  • 新增 MySQL/MongoDB Connector IT 类、Maven Failsafe 配置和可复用 GitHub Actions 工作流。
  • 集成测试连接配置已清理为无敏感值模板;运行时必须通过 CI Secrets 或环境变量注入。
  • 未把 feature-it 中含敏感配置的历史提交合入当前分支。

本地验证

  • JDK 17.0.20.1。
  • MySQL:mvn --batch-mode --offline -pl connectors/mysql-connector -am -Dmaven.gitcommitid.skip=true -Djacoco.skip=true test
    • exit 0;15 个模块成功;MysqlConnectorTest 17/17 通过;新增 MySQL IT 源编译成功。
  • MongoDB:mvn --batch-mode -pl connectors/mongodb-connector -am -Dmaven.gitcommitid.skip=true -Djacoco.skip=true test
    • exit 0;6 个模块成功;MongoDB 模块 179/179 通过;新增 MongoDB IT 源编译成功。
  • git diff --check:exit 0。
  • 配置敏感模式扫描:无命中。

残留风险

  • 完整 IT 依赖 CI 自托管 runner、数据库环境和 Secrets,结果需继续观察。
  • feature-it 曾出现硬编码凭据;相关凭据应由人工轮换,当前 PR 未携带其历史提交。
  • Java 版本与 tapdata-common 测试依赖调整扩大了兼容性范围,需重点审查。
  • 合并与 Jira 完成态流转由人工执行。

关联:TAP-12729

@HarsenLin HarsenLin changed the title fix(mysql): add FROM for large batch reads fix(mysql): add FROM and connector integration tests Aug 28, 2026
@HarsenLin HarsenLin closed this Aug 29, 2026
@HarsenLin
HarsenLin deleted the fix/TAP-12729-mysql-batch-read-sql-v2 branch August 29, 2026 14:51
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.

1 participant