feat!: reduce default max_unacked_bytes to 5MiB

[quettabit]

No description provided.

[greptile-apps]

Greptile Overview

Greptile Summary

Reduces the default max_unacked_bytes from 10MiB to 5MiB for both ProducerConfig and AppendSessionConfig, halving the default memory budget for unacknowledged data in-flight. This is a breaking change as it alters default behavior for users who rely on the implicit default.

• Default max_unacked_bytes changed from 10 * ONE_MIB to 5 * ONE_MIB in ProducerConfig::default() and AppendSessionConfig::default()
• Doc comments updated consistently in both locations to reflect the new 5MiB default
• No stale references to the old 10MiB default found anywhere in the codebase (examples, tests, docs)
• The with_max_unacked_bytes setter and its 1MiB minimum validation remain unchanged

Confidence Score: 5/5

• This PR is safe to merge — it is a minimal, well-contained default value change with no logic or behavioral side effects.
• The change is purely a constant default value reduction in two parallel config structs. Doc comments are updated consistently. No stale references to the old value exist. The validation logic and all other code paths remain untouched. There are no test failures, logic errors, or security concerns.
• No files require special attention.

Important Files Changed

Filename	Overview
src/producer.rs	Default max_unacked_bytes reduced from 10 * ONE_MIB to 5 * ONE_MIB in ProducerConfig::default(), with matching doc comment update. No issues found.
src/session/append.rs	Default max_unacked_bytes reduced from 10 * ONE_MIB to 5 * ONE_MIB in AppendSessionConfig::default(), with matching doc comment update. No issues found.

Flowchart

flowchart TD
    A[ProducerConfig::default] -->|"max_unacked_bytes = 5 MiB"| B[AppendPermits::new]
    C[AppendSessionConfig::default] -->|"max_unacked_bytes = 5 MiB"| B
    B -->|Creates Semaphore with byte budget| D[Backpressure Control]
    D -->|acquire permits before submit| E[Submit AppendInput / AppendRecord]
    E -->|Ack received| F[Release permits]
    F --> D

Loading

_{Last reviewed commit: d33f3cc}