[jira] [Commented] (ACCUMULO-3577) VolumeManager FileType needs unit tests

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

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

Christopher Tubbs commented on ACCUMULO-3577:
---------------------------------------------

To be clear, the bug looks like it would appear for any FileType, not just tables. Example: a volume called {{hdfs://nn1/walkerTexasRanger}} in the case of {{/wal}} write-ahead log file types.

> VolumeManager FileType needs unit tests
> ---------------------------------------
>
>                 Key: ACCUMULO-3577
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3577
>             Project: Accumulo
>          Issue Type: Bug
>            Reporter: Christopher Tubbs
>             Fix For: 1.7.0, 1.6.3
>
>
> The FileType enum in VolumeManager has some behavior which should be unit tested.
> In particular, it should check that the retrieval (or removal) of volumes from a path works correctly.
> At first glance, it looks to me like it is broken, because it is looking for the first occurrence of a directory which starts with {{/tables}}, in the case of tables. However, that will break if the volume itself had a path with {{/tables}} in it (example: {{hdfs://nn1/tablesorting-application/accumulo}}, which would have the directory {{hdfs://nn1/tablesorting-application/accumulo/tables/...}})



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