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: Add Noop processing strategy #376

Merged

Conversation

evanpurkhiser
Copy link
Member

This can be useful when you consume a message and do not wish to commit he offset back to Kafka [1].

[1]: The sentry uptime-checker project has this requirement as it is using Kafka as a mechanism to store configurations. By using log compaction we never actually want to commit a log offset, and want to read the entire log every time.

@evanpurkhiser evanpurkhiser requested review from a team as code owners July 1, 2024 19:44
@evanpurkhiser evanpurkhiser requested review from a team and lynnagara July 1, 2024 19:44
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-add-noop-processing-strategy branch 2 times, most recently from 6b20fd5 to c0afd21 Compare July 1, 2024 20:18
This can be useful when you consume a message and do not wish to commit
he offset back to Kafka [1].

[1]: The sentry uptime-checker project has this requirement as it is
     using Kafka as a mechanism to store configurations. By using log
     compaction we never actually want to commit a log offset, and want
     to read the entire log every time.
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-add-noop-processing-strategy branch from c0afd21 to 33caa04 Compare July 1, 2024 21:00
Copy link
Member

@lynnagara lynnagara left a comment

Choose a reason for hiding this comment

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

thanks!

@evanpurkhiser evanpurkhiser merged commit ab87c3a into main Jul 1, 2024
11 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/feat-add-noop-processing-strategy branch July 1, 2024 22:58
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.

2 participants