search

sirensolutions / siren-join

[This is the old, single node version for Elasticsearch 2.x, see the latest "Siren Federate" plugin for distributed Elasticsearch 5.x and 6.x capabilities]
http://siren.io
GNU Affero General Public License v3.0
183 stars 60 forks source link

Integrate a circuit breaker in the terms set computation #29

Closed rendel closed 8 years ago

rendel commented 8 years ago

Currently, if the user does not set a term limit, and if the term set is too big for the available memory, this could lead to a OOME. We could add a circuit breaker to protect against this.

jccq commented 8 years ago

EE feature possibly?

On Fri, Feb 12, 2016 at 11:15 AM, Renaud notifications@github.com wrote:

Closed #29 https://github.com/sirensolutions/siren-join/issues/29 via 152655d https://github.com/sirensolutions/siren-join/commit/152655dd21d70535a077f7de13c30e12ac3a3775 .

— Reply to this email directly or view it on GitHub https://github.com/sirensolutions/siren-join/issues/29#event-548397827.

Giovanni Tummarello CEO - SIREn Solutions

jccq commented 8 years ago

EE stuff? On 12 Feb 2016 11:15 a.m., "Renaud" notifications@github.com wrote:

Closed #29 https://github.com/sirensolutions/siren-join/issues/29 via 152655d https://github.com/sirensolutions/siren-join/commit/152655dd21d70535a077f7de13c30e12ac3a3775 .

— Reply to this email directly or view it on GitHub https://github.com/sirensolutions/siren-join/issues/29#event-548397827.