[ACCUMULO-2817] Add offset and limit arguments to byte array Encoder.decode method

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.7.0
• Component/s: client
• Labels: newbie
• Assignee: Matt Dailey
• Reporter: Josh Elser
• Created: 15/May/14 22:28
• Updated: 23/Jun/16 19:17
• Resolved: 26/Feb/15 03:18

Description

Similar to ACCUMULO-2445, but presently the encoder only works on complete byte arrays. This forces an extra copy of the data when it is located in an array that contains other information (e.g. a composite key).

It would be nice to be able to provide offset and length arguments to Encoder.decode so that users can avoid the additional arraycopy.

Changing to a ByteBuffer instead of byte array argument would also be acceptable, but more churn on the API that, unless it's happening globally, I would rather avoid. It would also incur the penalty for that extra Object, which while minimal alone, could be significant if decoding every value in a table, for example.

Issue Links

• breaks: ACCUMULO-3789 Code compiled against 1.6 lexicoders can't run in 1.7
• is related to: ACCUMULO-4352 Lexicoder decode(byte[], offset, length) into public API
• supercedes: ACCUMULO-3520 All Lexicoders should have unit tests