Skip to content

Feature/sql server to mysql overview#66

Closed
alswl wants to merge 175 commits intoalibaba:masterfrom
alswl:feature/sql-server-to-mysql-overview
Closed

Feature/sql server to mysql overview#66
alswl wants to merge 175 commits intoalibaba:masterfrom
alswl:feature/sql-server-to-mysql-overview

Conversation

@alswl
Copy link
Copy Markdown
Collaborator

@alswl alswl commented Apr 2, 2018

#50 related.

申请开源通过了,这个 PR 先作为 Overview 来看,不作为正式提交。

核心改造分别是:

  • 格式化代码 Google Style / dos2unix(一时爽,火葬场~)
  • 使用 fat jar 模式
  • 使用 YAML 配置文件格式
  • 改造 Plugin 模式,将 Java 内编译改为反射获取 Java 类
  • 拆分 Unit Test / Integration Test
  • 重构 Oracle 的继承结构,使其开放 SQL Server / MySQL 接口
  • 支持 SQL Server Source
  • 支持 MySQL Source
  • 支持 SQL Server CDC Source 作为在线迁移
  • 支持 Canal Redis 格式的数据作为 MySQL -> SQL Server 在线迁移

@agapple 看看,我如何合并到上游,我目前想法是,将这个 PR 按照功能逐个拆开,多次 PR 提交。
或者我们讨论一下哪些功能需要合并上游,哪些放弃(这需要大规模修改代码,和主分支偏离会比较严重)。

dijingchao added 30 commits May 3, 2017 14:50
…xtractor' into feature/sql-server-support #ARCH-1337
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 2, 2018

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.

✅ alswl
❌ Administrator


Administrator 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.

@alswl
Copy link
Copy Markdown
Collaborator Author

alswl commented May 10, 2018

@agapple 沟通下来,我先解决冲突的问题。

dijingchao added 2 commits May 21, 2018 12:50
@alswl
Copy link
Copy Markdown
Collaborator Author

alswl commented Jun 21, 2018

解决了冲突的版本:#70

@alswl alswl closed this Jun 21, 2018
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.

2 participants