-
Notifications
You must be signed in to change notification settings - Fork 52
better multiset serialization / deserialization #51
Comments
Can this be changed without breaking backwards compatibility? Especially considering that values themselves may be of any type, including numbers, Objects, Strings... |
I'm not familiar w/ the intricacies of jackson serialization / deserialization. I'd be pretty surprised if there is widespread use of serializing Multisets (using the existing jackson library)... I would have guessed a lot of other people would be bringing up this exact issue if they were using it. Does jackson ever create hard-breaks in backwards compatibility? |
Typically the way changes like this are handled is by introducing a new on/off This might be something to bring up on dev mailing list, esp. since there may be multiple ways data could be structured. But I can definitely see why change would be useful here; it's just a question of how to best improve the situation. |
Currently, multisets' serialization size is proportional to Multiset.size() as opposed to Multiset.entrySet().size().
For example, if I have do Multiset.setCount("my_key", 1000), "my_key" is repeated 1000 times in the output. It would be much more appropriate if it behaved liked Map<String, Integer>.
The text was updated successfully, but these errors were encountered: