[
https://issues.apache.org/jira/browse/ACCUMULO-3920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14624696#comment-14624696
]
ASF GitHub Bot commented on ACCUMULO-3920:
------------------------------------------
Github user keith-turner commented on the pull request:
https://github.com/apache/accumulo/pull/40#issuecomment-120941438
I am thinking we can drop `IntersectingIteratorTest.testWithBatchScanner()`
because `ExamplesIT.testShardedIndex()` will test IntersectingIter using a real
batch scanner in an IT. `IntersectingIteratorTest.testWithBatchScanner()` is
testing IntersectingIter using the mock batch scanner. Alternatively if we
want to treat this as a test of mock, we could just suppress warnings on the
test and add a comment stating the test is intended to test the mock batch
scanner.
> Deprecate mock and cease using it in our tests
> ----------------------------------------------
>
> Key: ACCUMULO-3920
> URL: https://issues.apache.org/jira/browse/ACCUMULO-3920
> Project: Accumulo
> Issue Type: Task
> Reporter: Christopher Tubbs
> Assignee: Christopher Tubbs
> Fix For: 1.8.0
>
>
> Numerous prior discussions have occurred, expressing a desire to deprecate
> (deprecate is not the same as remove) mock, so we can properly communicate
> its current status as a "dying" (poorly maintained, not kept up-to-date,
> lacking features) component of Accumulo. Some interest has been expressed in
> keeping it around, but no work has been put in to it to fix it's numerous
> divergences from the behavior of "real" Accumulo, add its missing features,
> or to improve its API.
> The task here is to:
> * Cease and desist using mock in our tests (replacing it with MAC, EasyMock,
> etc., as appropriate)
> * Mark mock {{@Deprecated}} until it is eventually removed in a major release
> Further discussion, if necessary, can be held later regarding its actual
> removal.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
