Skip to content
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

RFC: Add a serializable argument class in preparation for invocation across a process boundary #21928

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

schrockn
Copy link
Member

@schrockn schrockn commented May 17, 2024

Summary & Motivation

Preparing for user-space execution of AMP rules. This separates arguments into ones that might be serialized by the daemon and passed to user space. The other arguments (e.g. AssetGraphView) can be reconstructed in the user process

How I Tested These Changes

BK

@schrockn schrockn changed the title Add a serializable argument class in preparation for invocation across a process boundary RFC: Add a serializable argument class in preparation for invocation across a process boundary May 17, 2024
@schrockn schrockn force-pushed the FOU-204-make-asset-condition-evaluator-names-more-generic branch 2 times, most recently from 3dd1cdd to 2b15cac Compare May 17, 2024 16:52
Base automatically changed from FOU-204-make-asset-condition-evaluator-names-more-generic to master May 17, 2024 16:53
@schrockn schrockn force-pushed the FOU-204-make-asset-condition-evaluator-serializable-arguments branch from d66bc9b to d3235b7 Compare May 21, 2024 07:10
This was referenced May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant