You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Michael Miller (JIRA)" <ji...@apache.org> on 2017/05/02 14:18:04 UTC
[jira] [Commented] (ACCUMULO-3521) Unused Iterator-related classes
[ https://issues.apache.org/jira/browse/ACCUMULO-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15992981#comment-15992981 ]
Michael Miller commented on ACCUMULO-3521:
------------------------------------------
[~ctubbsii] With the updates in PR #237, Josh's in https://github.com/apache/accumulo/pull/247 and the test I added in https://github.com/apache/accumulo-examples/commit/bec846ce3dec6b594e5e54a1349a99f5202dd91b can I mark this resolved? I think the other methods in (4.) have been removed.
> 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
> Assignee: Michael Miller
> Fix For: 2.0.0
>
> Time Spent: 4.5h
> Remaining Estimate: 0h
>
> 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)