-
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
Add option to drop task or flow return values from memory #7174
Conversation
✅ Deploy Preview for prefect-orion ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Minor comments
I'm thinking that I should toggle |
75433c1
to
f2fedd0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both the proposed names and defaults make sense to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like it!
Adds a
cache_result_in_memory
option to flows and tasks to drop a result from memory instead of holding it. You can use this while disabling persistence which is the equivalent of saying "I shall not use this result downstream". By default, using it will enable persistence.Example
See the examples in the documentation
Checklist
<link to issue>
"fix
,feature
,enhancement