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

Ability to Marshal objects with memoized methods #10

Open
sferik opened this issue Dec 18, 2013 · 1 comment
Open

Ability to Marshal objects with memoized methods #10

sferik opened this issue Dec 18, 2013 · 1 comment

Comments

@sferik
Copy link
Contributor

sferik commented Dec 18, 2013

See discussion here: #4 (comment)

@mbj
Copy link
Collaborator

mbj commented Dec 18, 2013

This is an interesting question. If this would be memoization for adamantium I'd argue we'd not even try to serialize the cache state. As memoizable would also be used for objects with a mutable root state (lets call the memoized stuff child state), I'd say the child state must be serialized also.

I still consider memoization of return values that are calculated from possible mutable state a smell.

@dkubb If we add cache serialization to memoizable we should think about adamantium, and if it should serialize the memoization state, or not.

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

No branches or pull requests

2 participants