Skip to content

ra consumer may acquire+ release out of order#94

Merged
ninaiiad merged 1 commit into
mainfrom
ve/out_of_order
Jun 12, 2026
Merged

ra consumer may acquire+ release out of order#94
ninaiiad merged 1 commit into
mainfrom
ve/out_of_order

Conversation

@vladimir-ea

Copy link
Copy Markdown
Collaborator

the TCache RandomAccessConsumer may acquire cache entries out of order - this might cause read failures of mis-ordered reads since it cosidered them below its 'tail'.

e.g.:

  • network receives a new message: reserves tcache space for it, does a partial write
  • network receives another message: reserves tcache space for it, does a full write, emits a spine message
  • first message write completes, emits a spine message

the consumer reads the spine messages in order and so reads the higher sequenced tcache message first.

this change adds a window over which messages may be out of order.

@ninaiiad ninaiiad merged commit 85fc0d7 into main Jun 12, 2026
3 checks passed
@ninaiiad ninaiiad deleted the ve/out_of_order branch June 12, 2026 17:04
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