[CALCITE-1094] Replace AvaticaUtils.readFully(InputStream)'s use of ByteArrayOutputStream

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: avatica-1.7.0
• Component/s: avatica
• Assignee: Josh Elser
• Reporter: Josh Elser
• Created: 17/Feb/16 21:03
• Updated: 14/Mar/16 15:07
• Resolved: 03/Mar/16 17:34

Description

I remember reading a ticket a long time ago (ACCUMULO-1000) about how use of ByteArrayOutputStream can be sub-optimal for low-latency calls due to the synchronization around the write method.

While ByteArrayOutputStream gets the job done, an unsynchronized buffer is sufficient for reading the bytes of the HTTP body off of the wire and into a byte array. We don't need the synchronization guarantees that come with BAOS.