accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Miller (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-3521) Unused Iterator-related classes
Date Tue, 02 May 2017 14:18:04 GMT

    [ 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)

Mime
View raw message