[
https://issues.apache.org/jira/browse/SENTRY-2043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16249841#comment-16249841
]
Sergio Peña commented on SENTRY-2043:
-------------------------------------
SENTRY-999 is a refactoring code that made the KeyValue class case-sensitive.
SENTRY-2039 brought the case-insensitive back.
> 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)
