[jira] [Commented] (ACCUMULO-3400) VolumeManager should validate the choice that a VolumeChooser makes

["Christopher Tubbs (JIRA)" <ji...@apache.org>]

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

Christopher Tubbs commented on ACCUMULO-3400:
---------------------------------------------

Linking to ACCUMULO-3393, because the task was identified in the cleanup there.

> VolumeManager should validate the choice that a VolumeChooser makes
> -------------------------------------------------------------------
>
>                 Key: ACCUMULO-3400
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3400
>             Project: Accumulo
>          Issue Type: Task
>    Affects Versions: 1.6.1
>            Reporter: Christopher Tubbs
>             Fix For: 1.6.2, 1.7.0
>
>
> The return value of the VolumeChooser.choose() method should be verified to actually exist in the set of volumes available to the Accumulo instance.
> No VolumeChooser which exists in Accumulo code actually does this today (including the ones added by ACCUMULO-3177 and ACCUMULO-3178), but it's a good sanity check to have, to prevent future mistakes and programming errors. It would also be good to have, in order to enforce the API contract of the VolumeChooser, in case other implementations are provided by external developers.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)