-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CT-2377] [Spike] Performance improvement to memory footprint #7281
Comments
@nathaniel-may Thanks for opening! (I'd been meaning to) More context in this internal Slack thread As a general rule, should we be garbage-collecting the |
Spike resulted in a fix since it was easy-- but for future reference: The issue at hand seems to be that That said, we can control what happens when we read the values via the Instinctually, that feels like a best practice anyway, but I'd be curious to hear what others think. |
When a macro runs a call statement with
fetch_result
set toTrue
, that result may be held in memory longer than it needs to be.The text was updated successfully, but these errors were encountered: