[
https://issues.apache.org/jira/browse/ACCUMULO-4554?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15799752#comment-15799752
]
Christopher Tubbs commented on ACCUMULO-4554:
---------------------------------------------
I'm not sure why we even try to parse this at all... this class doesn't seem
useful or necessary for Accumulo. We can just print the version as-is. We don't
need to "recognize" it.
> Fail gracefully on an un-parseable version string
> -------------------------------------------------
>
> Key: ACCUMULO-4554
> URL: https://issues.apache.org/jira/browse/ACCUMULO-4554
> Project: Accumulo
> Issue Type: Improvement
> Reporter: Josh Elser
> Assignee: Josh Elser
> Priority: Minor
> Fix For: 2.0.0
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> Just had a delightfully odd failure case where every single IT that used a
> minicluster would timeout and fail.
> Turns out, if you happen to set a version on Accumulo (e.g. what I might do
> if I were testing locally and wanted to be isolated) which doesn't match what
> Version.java *wants* your version to be, an {{IllegalArgumentException}} is
> thrown which brings down all of the processes.
> The fun part is that this Version is used no-where, save the {{accumulo
> version}} command. We can and should fail gracefully -- there is no reason we
> should fail hard as we do now.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
