Project

General

Profile

Actions

Bug #8541

closed

[NodeManager] Use sys.exc_clear() to release exception tracebacks

Added by Peter Amstutz about 8 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assigned To:
-
Category:
-
Target version:
-
Story points:
-

Description

According to the discussion at http://cosmicpercolator.com/2016/01/13/exception-leaks-in-python-2-and-3/ exceptions can keep stack frames live far past when they are no longer needed, which may be the cause of our memory leaks.

Audit exception handling in node manager and use sys.exc_clear() discard exception tracebacks where possible.


Related issues

Related to Arvados - Idea #8543: [NodeManager] Don't use Futures when not expecting a replyResolvedPeter Amstutz03/04/2016Actions
Actions #1

Updated by Peter Amstutz about 8 years ago

  • Description updated (diff)
Actions #2

Updated by Brett Smith about 8 years ago

  • Target version set to Arvados Future Sprints
Actions #3

Updated by Peter Amstutz about 8 years ago

#8543 is another line of inquiry related to exceptions possibly sticking around for much longer than they should.

Actions #4

Updated by Brett Smith about 8 years ago

Our current thinking is that #8543 has improved the situation enough (based on RSS) that this should no longer be necessary, right?

Actions #5

Updated by Peter Amstutz about 8 years ago

Yes. While it's still possible it's holding onto some exceptions unnecessarily, it looks like #8543 solved the primary memory leak, so this ticket is low priority.

Actions #6

Updated by Peter Amstutz about 3 years ago

  • Status changed from New to Closed
Actions #7

Updated by Ward Vandewege almost 3 years ago

  • Target version deleted (Arvados Future Sprints)
Actions

Also available in: Atom PDF