Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

Add license so we can switch this project to public. #31

Merged
merged 1 commit into from
Mar 26, 2021

Conversation

fpacifici
Copy link
Contributor

No description provided.

@fpacifici fpacifici requested a review from a team March 26, 2021 01:51
@evanh
Copy link
Member

evanh commented Mar 26, 2021

Shouldn't this have the same license as Snuba?

@fpacifici
Copy link
Contributor Author

Not necessarily.
It does not directly depend on snuba. It only depends on kafka and postgres.
Now about the licensing.
This is not a sentry core business logic. It could potentially be used as a generic cdc solution outside of sentry as long as you want to stream data from postgres to kafka.
All sentry specific logic is by configuration which is not checked into the repo.
I think it is similar (as a use case) as https://github.com/getsentry/chartcuterie

Copy link
Member

@evanh evanh left a comment

Choose a reason for hiding this comment

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

Sounds good to me.

@fpacifici fpacifici merged commit 75446aa into master Mar 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants