You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Christopher Tubbs (JIRA)" <ji...@apache.org> on 2019/04/23 23:35:00 UTC

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

     [ https://issues.apache.org/jira/browse/ACCUMULO-4352?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Christopher Tubbs resolved ACCUMULO-4352.
-----------------------------------------
    Resolution: Won't Fix

If this is still requested, please create a new issue or pull request to fix it at https://github.com/apache/accumulo/issues
Since we now have default methods available to us, because 2.0 and later use Java 8 (or later), this may be easier to handle.

> 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: Shana Hutchison
>            Priority: Major
>
> 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
(v7.6.3#76005)