accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <>
Subject [jira] [Resolved] (ACCUMULO-4634) MockIteratorEnvironment implementation broken in mock accumulo framework
Date Mon, 15 May 2017 21:30:04 GMT


Josh Elser resolved ACCUMULO-4634.
    Resolution: Fixed

Thanks for the patch, Ivan!

> MockIteratorEnvironment implementation broken in mock accumulo framework
> ------------------------------------------------------------------------
>                 Key: ACCUMULO-4634
>                 URL:
>             Project: Accumulo
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 1.8.0
>            Reporter: Ivan Bella
>            Assignee: Ivan Bella
>              Labels: mock
>             Fix For: 1.8.2, 2.0.0
>   Original Estimate: 1h
>          Time Spent: 1.5h
>  Remaining Estimate: 0h
> The MockIteratorEnvironment was broken when the additional sampling methods were added.
 The isSamplingEnabled throws an unsupported operation exception instead of simply returning
false.  Also the getSamplerConfiguration should return null and the cloneWithSamplingEnabled
should throw a SampleNotPresentException instead an UnsupportedOperationException per the
documentation in the IteratorEnvironment interface.  This will allow its use to work smoothly
as before.  Granted this whole MockAccumulo mechanism is deprecated, however external projects
still have dependencies on it and hence we should not have broken its implementation.

This message was sent by Atlassian JIRA

View raw message