Make env_servlets cache an instance field instead of global.

[rohinb2]

We don't need this to be global, it's only used in the object store.

In general, minimizing the amount of global stuff we have referencing important objects is better to avoid weird memory issues.

[rohinb2]

• Use temporary executor that is cleaned up instead of a long lived ThreadPoolExecutor. #737
• Clean up an EnvServlet from global Runhouse daemon state if it dies unexpectedly. #734
• Make env_servlets cache an instance field instead of global. #736 👈
• Change ObjStore methods to call by env servlet name instead of actor. #733
• Change clear_local to be parallellized. #735
• Send the exception_as_str from EnvServlet as well, and use as a fallback if pickling fails. #732
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rohinb2 and the rest of your teammates on Graphite

[rohinb2]

Merge activity

• Apr 16, 2:02 PM EDT: @rohinb2 started a stack merge that includes this pull request via Graphite.
• Apr 16, 2:11 PM EDT: Graphite rebased this pull request as part of a merge.
• Apr 16, 2:12 PM EDT: @rohinb2 merged this pull request with Graphite.