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

feat(out_waterdrop): add basic waterdrop output plugin #508

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

raytung
Copy link
Member

@raytung raytung commented Apr 1, 2024

This is a super simple proof-of-concept implementation for Waterdrop as described in #500. We'll need a lot more work to get this up to scratch compared to the other output plugins but I don't want to make this PR even larger 😄

Copy link

This PR has been automatically marked as stale because it has been open 90 days with no activity. Remove stale label or comment or this PR will be closed in 30 days

@github-actions github-actions bot added the stale label Jul 30, 2024
@ashie ashie removed the stale label Jul 30, 2024
@cosmo0920
Copy link
Contributor

cosmo0920 commented Aug 1, 2024

This couldn't be included in v0.19.3 release. But can we mark as v0.20.0.rc milestone? This is because we have to seek successor of pure Ruby client for Kafka.

@ashie ashie added the enhancement Feature request label Aug 2, 2024
@ashie
Copy link
Member

ashie commented Aug 2, 2024

IMHO there is no problem merging this in any timing for make it as prototype of the new plugin and improve it step by step if we remove gem.add_dependency 'waterdrop', '~> 2.6'.
Merging PoC code for new feature but disable by default is a common practice in large OSS projects such as Chromium or Firefox.

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

Successfully merging this pull request may close these issues.

3 participants