Open jbparker opened 7 years ago
Hi @jbparker, I thought about that already and yes, adding compression to ProtoBuf makes sense as an option I think.
That being said, CacheManager also supports MS Bond now and there are already 3 variants of it.
I'd rather add a compressed
configuration flag instead of another type for a gz version. This would be a much larger change though.
Anyways, feel free to send a PR ;)
I'd rather add a compressed configuration flag instead of another type for a gz version. This would be a much larger change though.
I feel the same. I don't mind it being larger in scope to make the "right change". That said, I will try to keep the changes as narrow as possible.
Thanks for your time!
Similar to the JSON ability with
GzJsonCacheSerializer
, this feature adds the ability to compress ProtoBuf streams before performing serialization for distributed caching.I have code running as a custom serializer for this, but I cannot model it in the same manner of
GzJsonCacheSerializer
works as theProtoBufSerializer
class is internal.@MichaCo I wanted to check with you before making a PR to see if this would be desirable to have in the mainline source (or anything similar that would accomplish the same thing).
Thanks for this great library.