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

specify kernel-worker datatype limitations (errors, ...) precisely #2937

Open
warner opened this issue Apr 22, 2021 · 3 comments
Open

specify kernel-worker datatype limitations (errors, ...) precisely #2937

warner opened this issue Apr 22, 2021 · 3 comments
Assignees
Labels
SwingSet package: SwingSet xsnap the XS execution tool

Comments

@warner
Copy link
Member

warner commented Apr 22, 2021

As mentioned in #2936:

In the long run, we need to nail down what exactly we're willing to tolerate in console.log calls, what precisely is the set of values that might appear in deliveries and syscalls, and make sure all our manager-worker communication pathways can support that full set. Our deliveries and syscalls are fairly conservative, and were designed to be JSON serializable, so it's really console.log that's the outlier.

Originally posted by @dckc in #2936 (comment)

@warner warner added the SwingSet package: SwingSet label Apr 22, 2021
@warner warner self-assigned this Apr 22, 2021
@erights erights self-assigned this Apr 22, 2021
@erights
Copy link
Member

erights commented Apr 22, 2021

Hi @warner I'm joining you as co-assignee on this one.

@erights
Copy link
Member

erights commented Apr 22, 2021

See #2936 (comment)

@dckc dckc added the xsnap the XS execution tool label Apr 30, 2021
@dckc dckc changed the title specify kernel-worker datatype limitations precisely specify kernel-worker datatype limitations (errors, ...) precisely Apr 30, 2021
@dckc
Copy link
Member

dckc commented Apr 30, 2021

In particular, serializing errors according to best practices seems useful.
see #3000

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SwingSet package: SwingSet xsnap the XS execution tool
Projects
None yet
Development

No branches or pull requests

4 participants