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

Implement kafka consumer and sink #162

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jaysonsantos
Copy link

@jaysonsantos jaysonsantos commented Mar 9, 2020

This should make kafka sources able to send TSV files to S3

This should make kafka sources to send TSV files to S3
Copy link

@peel peel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey Jayson, thanks for your contribution. It looks good to me, it would be fantastic if you could apply suggestions below.

@peel peel added this to the Version 0.8.0 milestone Mar 16, 2020
@diseku
Copy link

diseku commented Jun 8, 2021

any update?

@paulboocock
Copy link
Contributor

Sorry for going silent on this issue for so long! We've not merged this in as running the Snowplow pipeline on top of Kafka isn't currently a high priority for us, and being transparent it likely won't be for the forseeable future. As such we don't have a particularly good and repeatable way to test pipelines running on top of Kafka which has prevented us merging this, once it's in we need to be able to maintain it which is why this is at a standstill. We'll update this issue should anything change on this stance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants