[jira] [Updated] (ACCUMULO-3577) VolumeManager FileType needs unit tests
[ https://issues.apache.org/jira/browse/ACCUMULO-3577?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christopher Tubbs updated ACCUMULO-3577: Fix Version/s: (was: 2.0.0) > 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 >Priority: Major > > 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 (v7.6.3#76005)
[jira] [Updated] (ACCUMULO-3577) VolumeManager FileType needs unit tests
[ https://issues.apache.org/jira/browse/ACCUMULO-3577?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Josh Elser updated ACCUMULO-3577: - Fix Version/s: (was: 1.7.0) 1.7.1 1.8.0 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.6.3, 1.8.0, 1.7.1 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)
[jira] [Updated] (ACCUMULO-3577) VolumeManager FileType needs unit tests
[ https://issues.apache.org/jira/browse/ACCUMULO-3577?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christopher Tubbs updated ACCUMULO-3577: Fix Version/s: 1.6.3 1.7.0 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)