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

TiCDC DML sink did not support pulsar+ssl:// #10602

Closed
kennytm opened this issue Feb 14, 2024 · 2 comments · Fixed by #10611
Closed

TiCDC DML sink did not support pulsar+ssl:// #10602

kennytm opened this issue Feb 14, 2024 · 2 comments · Fixed by #10611
Assignees
Labels
affects-7.5 area/ticdc Issues or PRs related to TiCDC. found/gs severity/major type/bug The issue is confirmed as a bug.

Comments

@kennytm
Copy link
Contributor

kennytm commented Feb 14, 2024

What did you do?

create a changefeed with

--sink-uri="pulsar+ssl://10.20.30.40:80/persistent://xxxx/yyyy/zzzz?protocol=canal-json"

similar to what has been documented in https://docs.pingcap.com/tidb/stable/ticdc-sink-to-pulsar#ticdc-authentication-and-authorization-for-pulsar

What did you expect to see?

Changefeed created successfully

What did you see instead?

Error: [CDC:ErrSinkURIInvalid]the sink scheme (pulsar+ssl) is not supported

Versions of the cluster

Upstream TiDB cluster version (execute SELECT tidb_version(); in a MySQL client):

(irrelevant)

Upstream TiKV version (execute tikv-server --version):

(irrelevant)

TiCDC version (execute cdc version):

Release Version: v7.5.0
Git Commit Hash: 8c006b38bdc304201d646c35c993156abcc2b845
Git Branch: heads/refs/tags/v7.5.0
UTC Build Time: 2023-11-20 05:31:19
Go Version: go version go1.21.3 linux/amd64
Failpoint Build: false
@kennytm kennytm added type/bug The issue is confirmed as a bug. area/ticdc Issues or PRs related to TiCDC. found/gs labels Feb 14, 2024
@kennytm kennytm changed the title DML sink did not support pulsar+ssl:// sink TiCDC DML sink did not support pulsar+ssl:// Feb 14, 2024
@kennytm
Copy link
Contributor Author

kennytm commented Feb 14, 2024

case sink.PulsarScheme:

This line is missing the case for sink.PulsarSSLScheme.

@fubinzh
Copy link

fubinzh commented Feb 19, 2024

/severity major

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-7.5 area/ticdc Issues or PRs related to TiCDC. found/gs severity/major type/bug The issue is confirmed as a bug.
Projects
3 participants