[infinispan-dev] Allocation costs of TypeConverterDelegatingAdvancedCache
mudokonman at gmail.com
Wed May 31 15:05:45 EDT 2017
Let me explain why it is there first :) This class was added for two main
reasons: as a replacement for compatibility and for supporting equality of
byte object. What this class does is at the user side is box the given
arguments (eg. byte -> WrappedByteArray) then the cache only ever deals
with the boxed types and then does the unboxing for any values that are
There are some exceptions with how the boxing/unboxing works for both cases
such as Streams and Indexing which have to rebox the data to work properly.
But the cost is pretty minimal.
While compatibility is always on or off unfortunately anyone can pass in a
byte at any point for a key or value. So we need to have these wrappers
there to make sure they work properly.
We could add a option to the cache, which people didn't show interest
before, to have a cache that doesn't support byte or compatibility. In
this case there would be no need for the wrapper.
By using the wrapper around the cache, compatibility becomes quite trivial
since we just need a converter in the wrapper and it does everything else
for us. My guess is the new encoding changes will utilize these wrapper
classes as well as they are quite easy to plug in and have things just work.
With the rewrite for eviction we lost the ability to use custom Equality in
the data container. The only option for that is to wrap a byte to provide
our own equality. Therefore the wrapper does this conversion for us by
converting between automatically.
On Wed, May 31, 2017 at 2:34 PM Sanne Grinovero <sanne at infinispan.org>
> Hi all,
> I've been running some benchmarks and for the fist time playing with
> Infinispan 9+, so please bear with me as I might shoot some dumb
> questions to the list in the following days.
> The need for TypeConverterDelegatingAdvancedCache to wrap most
> operations - especially "convertKeys" - is highlighet as one of the
It should be wrapping every operation pretty much.
Unfortunately the methods this hurts the most are putAll, getAll etc as
they have to not only box every entry but copy them into a new collection
as you saw in "convertKeys". And for getAll it also has to unbox the return
We could reduce allocations in the collection methods by not creating the
new collection until we run into one key or value that required
boxing/unboxing. This would still require fully iterating over the
collection at best case. This should perform well in majority of cases as I
would expect all or almost all entries either require or don't require the
boxing. The cases that would be harmed most would be ones that have a
sparse number of entries that require boxing.
> high allocators in my Search-centric use case.
> I'm wondering:
> A - Could this implementation be improved?
Most anything can be improved :) The best way would be to add another knob.
B - Could I bypass / disable it? Not sure why it's there.
There is no way to bypass currently. Explained above.
> infinispan-dev mailing list
> infinispan-dev at lists.jboss.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the infinispan-dev