Angelin01 / Angelin01/pod-director

Optimize patch operations

Open
#32 0 comments 0 reactions 0 assignees View on GitHub
good first issue maintenance
Dominant language
Rust
Stars
3
Forks
0
PR merge metrics
No merged PRs in 30d

Description

There are three simple optimizations:
- Replace the HashMap in the configuration for a BTreeMap for a slightly reduced memory footprint.
- For the patches, when can aggregate them into a single operation instead of multiple operations
- This is specially useful if the pod has no nodeSelector or tolerations at all, since we completely avoid a loop
- For the empty object or empty list, the behavior is similar to the one above

This could easily be benchmarked in a test or two, to validate the results

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.