Disable Real Memory Circuit Breaker by default and deprecate it#112115
Disable Real Memory Circuit Breaker by default and deprecate it#112115original-brownbear wants to merge 3 commits intoelastic:mainfrom original-brownbear:flip-default-rmcb
Conversation
As discussed, deprecating the setting and flipping it to `false` by default on account of the progress we made that makes this breaker hurt more than it helps.
|
Documentation preview: |
|
Pinging @elastic/es-core-infra (Team:Core/Infra) |
|
I understand the motivation of this change and I agree with it. Still I am concerned that some parts of the code rely on the real memory circuit breaker in order to avoid OOMs. This change will leave those cases totally out of coverage so I think we need to provide some other mechanism in order to have some protection against OOMs. The cases I am aware of: This is pretty important and the reason we increase the max number of buckets from 10k to 65k as it will give the protection in case we will be crashing too much data on heap on the coordinator node during reduction. Similar to the above but in the data node. There are some objects in the aggregation framework that are not memory tracked. That was added recently when we notice that building global ordinals can hit OOMs in clusters with small heap. That was added recently when we notice that building the filters on a filters aggregator can hit OOMs in clusters with small heap. |
As discussed, deprecating the setting and flipping it to
falseby default on account of the progress we made that makes this breaker hurt more than it helps.