Skip to content

Consider max unconfirmed messages in dynamic batch configuration #773

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

Merged
merged 1 commit into from
Jun 13, 2025

Conversation

acogoluegnes
Copy link
Contributor

A small value for max unconfirmed messages can impact the dynamic batch mechanism. This commit sets the min batch size to half the max unconfirmed messages value if it is less than the configured batch size.

References #757

A small value for max unconfirmed messages can impact the dynamic batch
mechanism. This commit sets the min batch size to half the max
unconfirmed messages value if it is less than the configured batch size.

References #757
@acogoluegnes acogoluegnes added this to the 1.2.0 milestone Jun 13, 2025
@acogoluegnes acogoluegnes added the refactoring Internal refactoring label Jun 13, 2025
@acogoluegnes acogoluegnes merged commit ce66ebb into main Jun 13, 2025
3 checks passed
@acogoluegnes acogoluegnes deleted the configure-dynamic-batch-with-max-unconfirmed branch June 13, 2025 15:23
github-actions bot pushed a commit that referenced this pull request Jun 13, 2025
…x-unconfirmed

Consider max unconfirmed messages in dynamic batch configuration
@pvaiko
Copy link

pvaiko commented Jun 17, 2025

Hi @acogoluegnes, thanks for the update, just to confirm, the snapshot would be com.rabbitmq:stream-client:1.2.0-SNAPSHOT from this repo: https://rabbitmq.github.io/rabbitmq-stream-java-client/snapshot/htmlsingle/#snapshots ? The problem may be on my end, but I wasn't able to download it.

@acogoluegnes
Copy link
Contributor Author

acogoluegnes commented Jun 17, 2025

This is the correct link the snapshot repository. I updated the project to use Sonatype's central portal last week, including for snapshots, but it seems it is down today, sorry about this.

@acogoluegnes
Copy link
Contributor Author

@pvaiko The snapshot repository is available again.

@pvaiko
Copy link

pvaiko commented Jun 18, 2025

@acogoluegnes thank you very much, just tested, works as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Internal refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants