AlexCouch / jewl-go

jewl-go is a go friendly performance analysis library
1 stars 1 forks source link

Decouple `RecorderCache` from `MsgPack` #6

Open AlexCouch opened 9 months ago

AlexCouch commented 9 months ago

Just like in Issue #2, we need to decouple the RecorderCache from Msgpack such that user/config schema can customize how the cache is marshalled/demarshalled.

A point of contention I have about this is how useful it would be to customize the RecordeCache since that is an internal function, and not something that users would likely need to customize. However, I expect that the cache may be expanded upon in the future, so I will keep this here in the event that it is need.

This may be closed by the time the 1.0-rewrite branch is finished without this being implemented. If it turns out that this is not really needed, we can close it. If we end up reversing that decision, we can always reopen this issue.