[jira] [Commented] (ACCUMULO-1005) Authorizations and ColumnVisibility API should not accept Charset param

["Hudson (JIRA)" <ji...@apache.org>]

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

Hudson commented on ACCUMULO-1005:
----------------------------------

Integrated in Accumulo-1.5-Hadoop-2.0 #100 (See [https://builds.apache.org/job/Accumulo-1.5-Hadoop-2.0/100/])
    ACCUMULO-1005 Applying patch contributed by Tim Reardon to remove Charset options from Authorizations and ColumnVisibility, with minor edits from John Vines and minor documentation additions and cleanup from myself (Revision 1479039)

     Result = SUCCESS
ctubbsii : 
Files : 
* /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/security/Authorizations.java
* /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/security/ColumnVisibility.java
* /accumulo/branches/1.5/core/src/test/java/org/apache/accumulo/core/security/AuthorizationsTest.java
* /accumulo/branches/1.5/core/src/test/java/org/apache/accumulo/core/security/ColumnVisibilityTest.java
* /accumulo/branches/1.5/core/src/test/java/org/apache/accumulo/core/security/VisibilityEvaluatorTest.java

                
> Authorizations and ColumnVisibility API should not accept Charset param
> -----------------------------------------------------------------------
>
>                 Key: ACCUMULO-1005
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1005
>             Project: Accumulo
>          Issue Type: Bug
>            Reporter: Christopher Tubbs
>            Assignee: Tim Reardon
>             Fix For: 1.5.0
>
>         Attachments: ACCUMULO-1005_deprecate.patch, ACCUMULO-1005.patch
>
>
> The Charset parameter was added to the public API for ACCUMULO-241. However, this intermingles internal serialization/comparison implementation, and the semantics of the public API.
> The Charset parameter effectively instructs Accumulo how to serialize the object. This can break the comparison with what is stored in the table and is an unnecessary breakage.
> In the public API, we should only accept Strings, and allow any valid java String. Internally, the serialization of these should consistently be UTF-8.

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