Skip to content

[DNM] changefeedccl: implement gocloud.dev/pubsub sink#43084

Draft
ajwerner wants to merge 1 commit intocockroachdb:masterfrom
ajwerner:ajwerner/implement-gocloud-pubsub
Draft

[DNM] changefeedccl: implement gocloud.dev/pubsub sink#43084
ajwerner wants to merge 1 commit intocockroachdb:masterfrom
ajwerner:ajwerner/implement-gocloud-pubsub

Conversation

@ajwerner
Copy link
Copy Markdown
Contributor

This is a very WIP PR to implement a changefeed sink for the generic pubsub API
exposed by gocloud.dev/pubsub. It does not currently deal with lack of ordering
for the nemesis test.

Fixes #36982.

Release note (enterprise change): Add support for gcp pubsub as a CHANGEFEED
sink.

This is a WIP PR to implement a changefeed sink for the generic pubsub API
exposed by gocloud.dev/pubsub. It does not currently deal with lack of ordering
for the nemesis test.

Fixes cockroachdb#36982.

Release note (enterprise change): Add support for gcp pubsub as a CHANGEFEED
sink.
@ajwerner ajwerner requested a review from danhhz December 10, 2019 19:04
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@glerchundi
Copy link
Copy Markdown

Any chance to push this forward @mwang1026? Would love to see this landing on the next release!

/cc @lopezator

@mwang1026
Copy link
Copy Markdown

I don't think this will make 20.2 (this October) but it's a strong consideration for 21.1 (next April). We'll update this thread when we have something more concrete and would love to include you as design partners!

@glerchundi
Copy link
Copy Markdown

glerchundi commented Sep 10, 2020

xrefing a comment I just made in the issue about ordering guarantees google is providing with pubsub: #36982 (comment).

@tbg tbg added the X-noremind Bots won't notify about PRs with X-noremind label May 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

X-noremind Bots won't notify about PRs with X-noremind

Projects

None yet

Development

Successfully merging this pull request may close these issues.

changefeedccl: consider adding additional sinks via go-cloud/pubsub

5 participants