Skip to content

[Issue 6957][pulsar-io-debezium] Support CDC Connector for SqlServer#7042

Closed
huangdx0726 wants to merge 13 commits into
apache:masterfrom
huangdx0726:fix6957
Closed

[Issue 6957][pulsar-io-debezium] Support CDC Connector for SqlServer#7042
huangdx0726 wants to merge 13 commits into
apache:masterfrom
huangdx0726:fix6957

Conversation

@huangdx0726

Copy link
Copy Markdown
Contributor

Fixes #6957

Motivation

Add debezium Sqlserver source

Modifications

Add debezium Sqlserver source

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

  • Added integration tests for end-to-end deployment with large payloads (10MB)
  • Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (yes / no)
  • The public API: (yes / no)
  • The schema: (yes / no / don't know)
  • The default values of configurations: (yes / no)
  • The wire protocol: (yes / no)
  • The rest endpoints: (yes / no)
  • The admin cli options: (yes / no)
  • Anything that affects deployment: (yes / no / don't know)

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
  • If a feature is not applicable for documentation, explain why?
  • If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@codelipenghui

Copy link
Copy Markdown
Contributor

@huangdx0726 Would you please help take a look at the failed CI tests? Looks related to this PR.

@huangdx0726

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@huangdx0726

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

2 similar comments
@huangdx0726

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@huangdx0726

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@codelipenghui

Copy link
Copy Markdown
Contributor

move to 2.7.0 first.

@codelipenghui codelipenghui modified the milestones: 2.6.0, 2.7.0 Jun 4, 2020
@jiazhai

jiazhai commented Jun 24, 2020

Copy link
Copy Markdown
Member

/pulsarbot run-failure-checks

2 similar comments
@sijie

sijie commented Jul 2, 2020

Copy link
Copy Markdown
Member

/pulsarbot run-failure-checks

@huangdx0726

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@codelipenghui

Copy link
Copy Markdown
Contributor

@huangdx0726 Could you please take a look at the failed CI tests? looks related to the maven poms.

@sijie

sijie commented Nov 12, 2020

Copy link
Copy Markdown
Member

@huangdx0726 Can you rebase it to the latest master?

@huangdx0726
huangdx0726 force-pushed the fix6957 branch 2 times, most recently from 1813676 to 4e65cdb Compare November 13, 2020 03:13
@huangdx0726

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

1 similar comment
@codelipenghui

Copy link
Copy Markdown
Contributor

/pulsarbot run-failure-checks

codelipenghui
codelipenghui previously approved these changes Nov 17, 2020
@codelipenghui

Copy link
Copy Markdown
Contributor

/pulsarbot run-failure-checks

@huangdx0726

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@huangdx0726

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

1 similar comment
@huangdx0726

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

tuteng
tuteng previously approved these changes Nov 20, 2020
@codelipenghui codelipenghui added this to the 2.8.0 milestone May 21, 2021
@codelipenghui

Copy link
Copy Markdown
Contributor

@huangdx0726 Could you please resolve the conflicts?

@codelipenghui codelipenghui modified the milestones: 2.8.0, 2.9.0 May 23, 2021
@eolivelli eolivelli modified the milestones: 2.9.0, 2.10.0 Oct 6, 2021
@codelipenghui codelipenghui modified the milestones: 2.10.0, 2.11.0 Jan 18, 2022
@congbobo184
congbobo184 dismissed stale reviews from tuteng and codelipenghui via d6a2b18 February 16, 2022 02:58
@jdunhamvrtx

Copy link
Copy Markdown

Hey Everyone,
Anything I can do to help this along? Seems it's really close.
-J

@huangdx0726

huangdx0726 commented May 15, 2022 via email

Copy link
Copy Markdown
Contributor Author

@codelipenghui codelipenghui modified the milestones: 2.11.0, 2.12.0 Jul 26, 2022
@tisonkun

Copy link
Copy Markdown
Member

I think this is resolved by #12256. Please open a new issue if it's not the case.

@tisonkun tisonkun closed this Nov 10, 2022
@tisonkun

Copy link
Copy Markdown
Member

Sorry that we don't give a review in time :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PulsarIO [Debezium] - Support MS SqlServer

8 participants