microsoft / microsoft/ebpf-for-windows

Change std::vector<map_cache_t> to std::map<uint64_t, map_cache_t>

Open
#211 1 comment 0 reactions 0 assignees View on GitHub
good first issue optimization triaged
Dominant language
C
Stars
3.6k
Forks
311
Avg merge
6d 10h
Merged PRs (30d)
21

Description

In api lib, currently std::vector is used to store the map cache. It will be more efficient to change it to std::map.

Contributor guide

Open the contributing guide

Research direction

Start by locating the API library code that stores the map cache as std::vector and trace all accesses to that cache. Check how the uint64_t key should map to each map_cache_t, then update the affected uses and run the relevant API library tests to confirm cache behavior remains correct.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
api
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.