[jira] [Commented] (ACCUMULO-3521) Unused Iterator-related classes

["Michael Miller (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ACCUMULO-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15945846#comment-15945846 ] 

Michael Miller commented on ACCUMULO-3521:
------------------------------------------

Can these be deprecated? OrIterator definitely has some bit rot.  At least if we deprecate it could bring to light if they are used to not.  

> Unused Iterator-related classes
> -------------------------------
>
>                 Key: ACCUMULO-3521
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3521
>             Project: Accumulo
>          Issue Type: Sub-task
>            Reporter: Christopher Tubbs
>             Fix For: 1.8.2, 2.0.0
>
>
> The following iterators and iterator-related utilities have no references at all in our code; minimally, they should have unit tests (this list does not include deprecated iterators):
> # OrIterator
> # FirstEntryInRowIterator (especially setNumScansBeforeSeek method is never called)
> # TypedValueCombiner (especially setLossyness method is never called)
> # IteratorUtil.getMaxPriority and .findIterator methods
> # StatsCombiner (available in the examples) does not use the setRadix method
> Note: these iterators may not be considered "public API", but might still be unsafe to remove the unused portions if somebody had used them.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)