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

Transaction batching Kafka producer #735

Merged
merged 18 commits into from
Jan 8, 2024
Merged

Conversation

bryanlb
Copy link
Contributor

@bryanlb bryanlb commented Nov 16, 2023

Summary

This introduces a new kafka producer that can batch bulk ingest request into larger transactional blocks using a single threaded consumer.

Also:

  • Adds supports support for Kafka additional properties on bulk ingest
  • Adds metrics for bulk ingest
  • Switches from blocking to virtual thread implementation for better throughput
  • Fixes java format to include support for new syntax (Bump google-java-format to 1.19.2 spotify/fmt-maven-plugin#185)
  • Refactors bulk ingest class structure to match other API endpoints

@bryanlb bryanlb added benchmark Apply to a PR to run JMH benchmarks and removed benchmark Apply to a PR to run JMH benchmarks labels Nov 16, 2023
@bryanlb bryanlb changed the title Bburkholder/exp transactions Transaction batching Kafka producer Dec 15, 2023
@bryanlb bryanlb force-pushed the bburkholder/exp-transactions branch from 52fac08 to faf5c7f Compare January 2, 2024 20:28
@bryanlb bryanlb force-pushed the bburkholder/exp-transactions branch from a93107c to af01334 Compare January 3, 2024 20:50
@bryanlb bryanlb force-pushed the bburkholder/exp-transactions branch from af01334 to 1d4aeef Compare January 3, 2024 21:04
@bryanlb bryanlb force-pushed the bburkholder/exp-transactions branch from 33fa764 to d92e759 Compare January 4, 2024 17:56
@bryanlb bryanlb force-pushed the bburkholder/exp-transactions branch from d92e759 to e2c2727 Compare January 4, 2024 18:12
@bryanlb bryanlb marked this pull request as ready for review January 5, 2024 18:48
@bryanlb bryanlb merged commit d99ae81 into master Jan 8, 2024
3 checks passed
@bryanlb bryanlb deleted the bburkholder/exp-transactions branch January 8, 2024 19:56
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