Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: jira@apache.org
Date: Tue, 5 Feb 2013 21:47:15 +0000 (UTC)
From: "Billie Rinaldi (JIRA)" <jira@apache.org>
To: notifications@accumulo.apache.org
Message-ID: <JIRA.12548353.1332869352745.249233.1360100835101@arcas>
In-Reply-To: <JIRA.12548353.1332869352745@arcas>
References: <JIRA.12548353.1332869352745@arcas>
Subject: [jira] [Commented] (ACCUMULO-498) Key.getColumnVisibility() should
 return a ColumnVisibility, not a Text
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Billie Rinaldi commented on ACCUMULO-498:
-----------------------------------------

Looks OK to me.  Adding a more informative javadoc as Josh suggests couldn't hurt.
                
> Key.getColumnVisibility() should return a ColumnVisibility, not a Text
> ----------------------------------------------------------------------
>
>                 Key: ACCUMULO-498
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-498
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: client
>    Affects Versions: 1.4.0, 1.3.5
>            Reporter: John Vines
>            Assignee: Luke Brassard
>            Priority: Trivial
>              Labels: newbie
>             Fix For: 1.5.0
>
>         Attachments: ACCUMULO-498.patch
>
>
> Currently getColumnVisibility() returns a Text. This doesn't make a lot of sense, since all other methods dealing with column visibilities use the ColumnVisibility object. We should use that instead.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira