-
Notifications
You must be signed in to change notification settings - Fork 69
Open
Labels
C-Moderate EffortShould take a moderate amount of time to address.Should take a moderate amount of time to address.L-ImportTo do with importing Swarm modules.To do with importing Swarm modules.S-ModerateThe fix or feature would substantially improve user experience.The fix or feature would substantially improve user experience.Z-ResearchThis issue requires inventing something new and/or studying research papers for clues.This issue requires inventing something new and/or studying research papers for clues.
Description
Depends on #2540 .
To generate the environment for a module, we use restrictEnv to take the runtime environment and restrict it to only those names which are supposed to be exported from the module. The resulting Map is much smaller, of course, but environments also use a tree structure to record the sequence of operations that was used to construct them, including deletion. So I am afraid that the tree structures for module environments are going to be very large --- they will mention everything that was in the environment before and then everything that was deleted.
We should (1) investigate whether this is, in fact, the case (and how big of a problem it is), and (2) think of ways to mitigate it, if it is a problem.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
C-Moderate EffortShould take a moderate amount of time to address.Should take a moderate amount of time to address.L-ImportTo do with importing Swarm modules.To do with importing Swarm modules.S-ModerateThe fix or feature would substantially improve user experience.The fix or feature would substantially improve user experience.Z-ResearchThis issue requires inventing something new and/or studying research papers for clues.This issue requires inventing something new and/or studying research papers for clues.