Skip to content

Conversation

@justonedev1
Copy link
Collaborator

@justonedev1 justonedev1 commented May 2, 2025

this is follow up on OCTRL-1003 with changes to kafka which I used to test metrics on STG and openstack. Just small tuneup of kafka-go producer, so it does not take 1s to send batch of messages which is incomplete (not 100 messages).

This branch is based on OCTRL-1003 (to distinguish metrics changes and kafka changes) so that one should be merged first to see real changes just in writer.go and writer_test.go

@justonedev1 justonedev1 requested a review from knopers8 as a code owner May 2, 2025 15:40
@justonedev1 justonedev1 force-pushed the kafka_adjustments branch from 7777e4f to 4b0a984 Compare May 7, 2025 10:32
@knopers8
Copy link
Collaborator

knopers8 commented May 9, 2025

could you rebase please?

@justonedev1 justonedev1 reopened this May 13, 2025
@justonedev1
Copy link
Collaborator Author

sorry for closing the PR, I am not sure how it happened :)

@justonedev1
Copy link
Collaborator Author

@knopers8 it should be rebased now

@knopers8 knopers8 merged commit 4af88a5 into master May 13, 2025
4 checks passed
@knopers8 knopers8 deleted the kafka_adjustments branch May 13, 2025 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants