You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Josh Elser (JIRA)" <ji...@apache.org> on 2014/02/22 03:35:21 UTC

[jira] [Commented] (ACCUMULO-1376) Shell should verify before dropping user

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

Josh Elser commented on ACCUMULO-1376:
--------------------------------------

[~chentecaliente], functionally, the patch changes look good. A couple of small things:

# What branch is this patch based on?
# Make sure you use (or follow) the [codestyle guidelines|http://accumulo.apache.org/source.html]. Most notably, 2spaces instead of a tab character.
# Could you add a test for this while you're making these changes? (1.5 ShellServerTest, 1.6.0+ ShellServerIT)

> Shell should verify before dropping user
> ----------------------------------------
>
>                 Key: ACCUMULO-1376
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1376
>             Project: Accumulo
>          Issue Type: Bug
>          Components: shell
>    Affects Versions: 1.5.0
>            Reporter: John Vines
>            Assignee: Billie Rinaldi
>            Priority: Trivial
>              Labels: patch
>         Attachments: ACCUMULO-1376.patch
>
>
> The shell checks before you drop a table. It should probably do the same for users.



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