Details
-
Bug
-
Status: Resolved
-
Critical
-
Resolution: Fixed
-
0.7
-
None
-
None
Description
private def byteBufferAllocate(size: Int): ByteBuffer = {
var buffer: ByteBuffer = null
try
catch
{ case e: OutOfMemoryError => throw new RuntimeException("OOME with size " + size, e) case e2 => throw e2 } buffer
}
This hides the fact that an Error occurred, and will likely result in some log handler printing a message, instead of exiting with non-zero status. Knowing how large the allocation was that caused an OOM is really nice, so I'd suggest logging in byteBufferAllocate and then re-throwing OutOfMemoryError