Skip to content

test(inkless:consolidation): adjust tests to address failures - #728

Open
viktorsomogyi wants to merge 1 commit into
mainfrom
svv/ts-unification-e2e-followup
Open

test(inkless:consolidation): adjust tests to address failures#728
viktorsomogyi wants to merge 1 commit into
mainfrom
svv/ts-unification-e2e-followup

Conversation

@viktorsomogyi

@viktorsomogyi viktorsomogyi commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Consolidation fetch requests segment.bytes minus max.message.bytes so one extra batch still fits the follower segment. Default max.message.bytes is 1048588 (1 MiB plus record overhead), so the old 1 MiB setting left a 1-byte fetch and starved the fetcher.

2 MiB leaves about 1 MiB of fetch budget. segment.ms still rolls on time.

@viktorsomogyi
viktorsomogyi force-pushed the svv/ts-unification-delete-records-e2e branch from 3f7ea8d to 270a1cc Compare August 14, 2026 10:13
@viktorsomogyi
viktorsomogyi force-pushed the svv/ts-unification-e2e-followup branch from d2c5d85 to 50eead0 Compare August 14, 2026 12:00
@viktorsomogyi
viktorsomogyi changed the base branch from svv/ts-unification-delete-records-e2e to main August 14, 2026 12:02
@viktorsomogyi
viktorsomogyi marked this pull request as ready for review August 14, 2026 12:02
Consolidation fetch requests segment.bytes minus max.message.bytes so
one extra batch still fits the follower segment. Default max.message.bytes
is 1048588 (1 MiB plus record overhead), so the old 1 MiB setting left a
1-byte fetch and starved the fetcher.

2 MiB leaves about 1 MiB of fetch budget. segment.ms still rolls on time.
@viktorsomogyi
viktorsomogyi force-pushed the svv/ts-unification-e2e-followup branch from 50eead0 to 9532a7f Compare August 14, 2026 14:03
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.

1 participant