[REEF-1539] Fix memory leak in InteropUtil.cpp

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Component/s: REEF Bridge
• Assignee: Patrick Chung
• Reporter: Patrick Chung
• Created: 23/Aug/16 00:35
• Updated: 24/Aug/16 00:25
• Resolved: 24/Aug/16 00:23

Description

ManagedByteArrayFromJavaByteArray in Org.Apache.REEF.Bridge.InteropUtil.cpp uses JNI GetByteArrayElements() but does not release the bytes. This causes the driver to slowly leak memory over time, which is exacerbated if the driver message is large.

JNI documentation recommends that Release<PrimitiveType>ArrayElements() be called after Get<PrimitiveTypeType>ArrayElements() to avoid leaking memory, since the returned array may be a copy of the Java array.