Keith Turner created ACCUMULO-4716:
--------------------------------------
Summary: Do not attempt to cache blocks over max array size
Key: ACCUMULO-4716
URL: https://issues.apache.org/jira/browse/ACCUMULO-4716
Project: Accumulo
Issue Type: Bug
Reporter: Keith Turner
One might think you could do {{new byte\[Integer.MAX_VALUE\]}} in Java, but as
I found when looking into ACCUMULO-4708 *suprise* you can't.
According to the [stack overflow post |https://stackoverflow.com/a/8381338]
{{new byte\[Integer.MAX_VALUE - 8\]}} should be safe. I was able to do up to
{{new byte\[Integer.MAX_VALUE - 2\]}} in my local testing.
When Accumulo caches a block it reads it into a byte array. For example this
code in
[CachableBlockFile.java|https://github.com/apache/accumulo/blob/rel/1.8.1/core/src/main/java/org/apache/accumulo/core/file/blockfile/impl/CachableBlockFile.java#L345]
does this when {{_currBlock.getRawSize() <= cache.getMaxSize()}}.
This code should ensure the size is less than {{min( cache.getMaxSize(),
MAX_ARRAY_SIZE)}} inorder to read it into a byte array.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
