accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-4554) Fail gracefully on an un-parseable version string
Date Wed, 04 Jan 2017 23:52:58 GMT


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:
>             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 *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

View raw message