[APEXMALHAR-2178] Unnecessary byte array copy in KryoSerializableStreamCodec

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Labels: newbie
• Assignee: Unassigned
• Reporter: Vlad Rozov
• Created: 09/Aug/16 01:55
• Updated: 16/Nov/16 09:13

Description

  public Slice toByteArray(T info)
  {
    ByteArrayOutputStream os = new ByteArrayOutputStream();
    Output output = new Output(os);

    kryo.writeClassAndObject(output, info);
    output.flush();
    return new Slice(os.toByteArray(), 0, os.toByteArray().length);
  }

It is not necessary to call os.toByteArray().length as it will result in duplicate copy of the byte array.

Issue Links

• is related to: APEXCORE-502 Unnecessary byte array copy in DefaultKryoStreamCodec.toByteArray
• links to: GitHub Pull Request #498