You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@sentry.apache.org by "Sergio Peña (JIRA)" <ji...@apache.org> on 2017/11/13 17:19:03 UTC
[jira] [Created] (SENTRY-2043) Understand the reason of the
KeyValue class design
Sergio Peña created SENTRY-2043:
-----------------------------------
Summary: 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)