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(events): forward the tenant configuration as part of the kafka message #4986

Open
lsampras opened this issue Jun 13, 2024 · 1 comment · May be fixed by #5224
Open

feat(events): forward the tenant configuration as part of the kafka message #4986

lsampras opened this issue Jun 13, 2024 · 1 comment · May be fixed by #5224
Assignees

Comments

@lsampras
Copy link
Member

In a previous issue we've added tenant id to kafka events so that it can be used in downstream clickhouse,

But due to recent changes the tenant_id won't be used as a unique identifier on clickhouse and instead we need to pass in the entire Tenant object to kafka store,

with this the underlying KafkaProducer will accept a Tenant as well and send the clickhouse_database property from it as a field.

@lsampras
Copy link
Member Author

This changes need to be added in the existing #4867 PR

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

Successfully merging a pull request may close this issue.

3 participants