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

Sergio Peña commented on SENTRY-2043:
-------------------------------------

FYI [~akolb] 

> Understand the reason of the KeyValue class design
> --------------------------------------------------
>
>                 Key: SENTRY-2043
>                 URL: https://issues.apache.org/jira/browse/SENTRY-2043
>             Project: Sentry
>          Issue Type: Task
>          Components: Sentry
>            Reporter: Sergio Peña
>
> The KeyValue class is used to split a key-value string that Sentry uses to 
> understand permissions. The community has some concern around this class 
> because a string parsing is done on the constructor and the equals() method 
> was case-insensitive before 1.7, then case-sensitive in 1.8 (caused 
> hbase-indexer test failures), then case-insensitive on 2.0.
> We should investigate if we can replace the KeyValue by a different more 
> common class, such as apache commons Pair, and if we can find the reason 
> about case sensitivity with other models.
> This is a task jira, and if we find improvements on the class or the code, 
> then we should create another JIRA to do that.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to