Skip to content

Kafka support and multitenant awareness#163

Open
michaelfortin wants to merge 2 commits into
timescale:mainfrom
Elucidia:mic/kafka-and-multitenant-awareness
Open

Kafka support and multitenant awareness#163
michaelfortin wants to merge 2 commits into
timescale:mainfrom
Elucidia:mic/kafka-and-multitenant-awareness

Conversation

@michaelfortin

Copy link
Copy Markdown

No description provided.

@michaelfortin michaelfortin requested a review from mostafa as a code owner June 3, 2026 16:01
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@mostafa

mostafa commented Jun 3, 2026

Copy link
Copy Markdown
Member

Hey @michaelfortin,

Thanks for your contribution. I love it.

Is this ready for review? I guess not yet due to the following:

  1. There seems to be merge conflicts that need to be resolved. I can't run tests because of it.
  2. CLA is not signed yet.
  3. No PR description.
  4. Everything is committed in two commits.
  5. Commits are not signed.
  6. I still don't know what the multi-tenancy is for.

BTW, I already skimmed the code and it looks good with some minor tweaks.

@michaelfortin

Copy link
Copy Markdown
Author

Hi @mostafa ,

For our usecase, we are ingesting logs from different clients. Multi tenancy is for preventing logs from different clients from correlating with each other.

Thanks for your feedback! I will clean up this PR in July when I'll have more time to work on this project.

@mostafa

mostafa commented Jun 20, 2026

Copy link
Copy Markdown
Member

@michaelfortin Thank you! 🙏 It's worth considering a few different ways of implementing multi-tenancy before settling on one implementation. Let's discuss multi-tenancy in a separate issue/PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants