[
https://issues.apache.org/jira/browse/ACCUMULO-4086?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15067153#comment-15067153
]
Christopher Tubbs commented on ACCUMULO-4086:
---------------------------------------------
I'm not so concerned about the {{PreferredVolumeChooser}}, because it's just
one possible implementation, which could be swapped out with a different
behavior. I'm slightly concerned about the {{PerTableVolumeChooser}} falling
back to a default. It should probably fall back to whatever the default value
is for {{table.volume.chooser}} (in {{DefaultConfiguration}}).
I'm more concerned about other places, like whatever {{VolumeManagerImpl}} is
doing, where we just don't have an appropriate scope available to configure for
that case. In these cases, if the situation warrants it, I think we should
think of it as a distinct scope, and do whatever it is we do for ACCUMULO-4085.
There may be some utility in specifying a failsafe chooser (in the case of an
actual failure... not just an overlooked scope which doesn't have a
configuration point), for example if the specified one for that scope cannot be
loaded from the classpath. However, at the risk of introducing an infinite
regress (what happens if the failsafe fails to load?), it might be better to
just fail until the config/classpath is fixed.
> Allow configurable failsafe volume choosing
> -------------------------------------------
>
> Key: ACCUMULO-4086
> URL: https://issues.apache.org/jira/browse/ACCUMULO-4086
> Project: Accumulo
> Issue Type: Sub-task
> Components: core
> Reporter: Christopher Tubbs
> Fix For: 1.8.0
>
>
> From parent issue:
> {quote}
> 3. In several places including {{PreferredVolumeChooser}},
> {{PerTableVolumeChooser}} and {{VolumeManagerImpl}}, the failsafe chooser is
> the {{RandomVolumeChooser}} which will include the instance volume that needs
> to be excluded. It would be useful to have a configurable failsafe in this
> situation.
> {quote}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
