Skip to content

perf: event in-place marking, VIS map reuse, EventManager pool, snapshot release#2845

Draft
pdrobnjak wants to merge 1 commit intoperf/pool-vis-occ-cms-fastpathfrom
perf/event-marking-vis-reuse-eventmgr-pool
Draft

perf: event in-place marking, VIS map reuse, EventManager pool, snapshot release#2845
pdrobnjak wants to merge 1 commit intoperf/pool-vis-occ-cms-fastpathfrom
perf/event-marking-vis-reuse-eventmgr-pool

Conversation

@pdrobnjak
Copy link
Contributor

Summary

  • Mark events in-place, reuse VIS maps, pool EventManager instances, and release snapshots eagerly

Stack

10/13 — depends on perf/pool-vis-occ-cms-fastpath

🤖 Generated with Claude Code

@github-actions
Copy link

github-actions bot commented Feb 11, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedFeb 11, 2026, 1:38 PM

@pdrobnjak pdrobnjak self-assigned this Feb 11, 2026
@pdrobnjak pdrobnjak force-pushed the perf/pool-vis-occ-cms-fastpath branch from d5ca49b to db03810 Compare February 11, 2026 13:19
@pdrobnjak pdrobnjak force-pushed the perf/event-marking-vis-reuse-eventmgr-pool branch from d6b8a16 to 30b77de Compare February 11, 2026 13:19
…hot release

- MarkEventsToIndex: modify events in-place instead of copying (~4.5 GB)
- NewEvent: pre-allocate attributes slice to exact size
- EventManager: sync.Pool for reuse across EVM snapshots (~2-3 GB)
- VIS Reset: clear() maps instead of make() via clone-on-store (~4 GB)
  - SetReadset/SetIterateset clone data before storing in txSlot
  - Reset() reuses existing map/slice capacity
- RevertToSnapshot: release CMS stores and EventManagers from
  discarded contexts back to sync.Pools (~5-8 GB)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@pdrobnjak pdrobnjak force-pushed the perf/event-marking-vis-reuse-eventmgr-pool branch from 30b77de to ba228a5 Compare February 11, 2026 13:37
@pdrobnjak pdrobnjak force-pushed the perf/pool-vis-occ-cms-fastpath branch from db03810 to 5732ec9 Compare February 11, 2026 13:37
@pdrobnjak pdrobnjak marked this pull request as draft February 11, 2026 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant