memoizable icon indicating copy to clipboard operation
memoizable copied to clipboard

Ability to Marshal objects with memoized methods

Open sferik opened this issue 12 years ago • 1 comments

See discussion here: https://github.com/dkubb/memoizable/pull/4#issuecomment-30849404

sferik avatar Dec 18 '13 15:12 sferik

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.

mbj avatar Dec 18 '13 16:12 mbj