elastic / elastic/ml-cpp

[ML] Include memory required for persistence in data frame analytics memory estimates

Open
#1,900 0 comments 0 reactions 0 assignees View on GitHub
:ml/DataFrameAnalysis
Dominant language
C++
Stars
157
Forks
67
Avg merge
12h 48m
Merged PRs (30d)
16

Description

The memory estimates used in data frame analytics should include an allowance for the extra memory required during persistence and restoration. Otherwise during persistence or restoration on a maxed-out node there is a risk that some process (not necessarily the job doing persistence or restoration) gets killed because there isn't enough memory to accommodate both the model and the extra data structures used during persistence and restoration.

It would be reasonable that this is just a simple multiplier of the estimated model size. If the main extra memory usage is in compressed form then maybe 10% would be enough, or maybe 10% plus a capped amount for a single uncompressed chunk, for example, `reported memory estimate = raw memory estimate * 1.1 + min(16MB, raw memory estimate)`.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.