Skip to content

[eno] Update input controller to use buffer for writes - #634

Merged
Ruinan Liu (ruinan-liu) merged 7 commits into
mainfrom
users/ruinanliu/inputBufferWrite
Aug 7, 2026
Merged

[eno] Update input controller to use buffer for writes#634
Ruinan Liu (ruinan-liu) merged 7 commits into
mainfrom
users/ruinanliu/inputBufferWrite

Conversation

@ruinan-liu

Copy link
Copy Markdown
Collaborator

On busy underlays, reduce write frequency and use buffer writes instead.

Adds a queue-depth gauge (eno_input_revision_write_buffer_depth), a
flush-error counter partitioned by op (eno_input_revision_write_buffer_flush_errors_total),
and a successful-flush counter (eno_input_revision_write_buffer_flush_total),
mirroring the existing ResourceSliceWriteBuffer metrics. These errors
retry silently today, so without this counter they're invisible.

Addresses review item 5 on PR #634.
Comment thread internal/flowcontrol/inputrevbuffer.go
Comment thread internal/flowcontrol/inputrevbuffer.go

@ruinan-liu Ruinan Liu (ruinan-liu) left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reply to comment

@ruinan-liu
Ruinan Liu (ruinan-liu) merged commit 8d8cc9f into main Aug 7, 2026
69 of 70 checks passed
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