accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ted Malaska (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-1933) Make unit on memory parameters case-insensitive
Date Thu, 02 Jan 2014 18:41:55 GMT

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

Ted Malaska commented on ACCUMULO-1933:
---------------------------------------

I don't think I can omit "B" and still be backwards compatible.  

If I leave "b" as is, then the code errors out currently.  Hmm

I think I have two options:
1. Support G, M, K, and B in all cases
2. Support G, M, and K in all cases but B only in upper case and add a clean error message
for b

What do you think.  I'm open to both.

> Make unit on memory parameters case-insensitive
> -----------------------------------------------
>
>                 Key: ACCUMULO-1933
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1933
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: shell
>    Affects Versions: 1.5.0
>            Reporter: Josh Elser
>            Priority: Minor
>              Labels: newbie
>
> {{config -t my_table -s table.split.threshold=2g}} fails with a non-intuitive error.
> {{config -t my_table -s table.split.threshold=2G}} succeeds
> It would be nice for both to work.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Mime
View raw message