[CASSANDRA-17128] JVMStabilityInspector.forceHeapSpaceOomMaybe should handle all non-heap OOMs rather than only supporting direct only

• Type: Improvement
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 4.1-alpha1, 4.1
• Component/s: Local/Other
• Change Category: Operability
• Complexity: Low Hanging Fruit
• Platform: All

Source Control Link

https://github.com/apache/cassandra/commit/8b11a22ce7011e98dd385b2cef793e5bd2f75c11

Test and Documentation Plan

tests

• Assignee: David Capwell
• Reporter: David Capwell

Authors

David Capwell

Reviewers

Caleb Rackliffe, Yifan Cai

• Created: 08/Nov/21 23:00
• Updated: 27/May/22 19:24
• Resolved: 09/Nov/21 01:54

Description

When an OOM is thrown we do normally want the JVM to halt (assuming JVM flags state to), but this only happens in very specific cases (see CASSANDRA-15214 for more details). Rather than relying on an allow list to convert to heap-oom, we should rely on an deny list; this way we handle every OOM thrown.

Issue Links

• links to: Circle CI: trunk
• links to: GH PR: trunk