You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Dylan Hutchison (JIRA)" <ji...@apache.org> on 2016/06/23 19:16:16 UTC

[jira] [Created] (ACCUMULO-4352) Lexicoder decode(byte[], offset, length) into public API

Dylan Hutchison created ACCUMULO-4352:
-----------------------------------------

             Summary: Lexicoder decode(byte[], offset, length) into public API
                 Key: ACCUMULO-4352
                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4352
             Project: Accumulo
          Issue Type: Improvement
          Components: client
            Reporter: Dylan Hutchison


I'm interested in calling {{decodeUnchecked(byte[], off, len)}} as public API, since it removes the need to copy a subarray out from a larger array.  Currently the method sits in the {{AbstractEncoder}} class and is {{protected}}.

There was a discussion on ACCUULO-2817 to make decodeUnchecked public and add it to the Lexicoder interface. The discussion involved a debate on binary compatibility and semvar.  I can't tell what the resolution was.

One option is to create a new interface extending Lexicoder with the new method decodeUnchecked. It would maintain binary compatibility, at the cost of adding another interface that exists only for binary compatibility reasons. A few other options were suggested.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)