-
Notifications
You must be signed in to change notification settings - Fork 97
feat: add CRaC priming support to powertools-kafka module #2145
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
base: main
Are you sure you want to change the base?
feat: add CRaC priming support to powertools-kafka module #2145
Conversation
Hi @kjswaruph. Thank you for the PR! I have assigned the PR to you and we will begin reviewing it and giving feedback soon. |
Hey @kjswaruph, thanks for opening this PR 🚀 I will review it in depth next week. In the meantime, let's try to increase the priming to the other serializers beyond JSON. I think we can leverage Avro Let's ignore Protobuf for now since it doesn't support something similar to |
Hey @kjswaruph, do you already know when you can work on this? Let me know if you need any help! |
Hi @phipag, sorry for responding late. I’ll update the PR with the Avro GenericRecord as recommended and aim to finish it by this weekend. If I encounter any issues or need clarification, I’ll reach out. Thanks again for your feedback! |
- Add maven test profile and classesloaded.txt for preloading - Add Crac dependency and update PowertoolsSerializer to register as Crac Resource - Add tests in PowertoolsSerializerTest to assert beforeCheckpoint and afterRestore hooks do not throw exception
2355544
to
46f66e3
Compare
|
@phipag I added the avro priming. Please review and tell me if any changes are required. |
Summary
Add Crac support for kafka module of powertools.
Changes
Issue number: #1999
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.