[
https://issues.apache.org/jira/browse/YARN-2818?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14202075#comment-14202075
]
Hudson commented on YARN-2818:
------------------------------
SUCCESS: Integrated in Hadoop-Hdfs-trunk #1926 (See
[https://builds.apache.org/job/Hadoop-Hdfs-trunk/1926/])
YARN-2818. Removed the now unnecessary user entity injection from Timeline
service given we now have domains. Contributed by Zhijie Shen. (vinodkv: rev
f5b19bed7d71979dc8685b03152188902b6e45e9)
*
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice/src/main/java/org/apache/hadoop/yarn/server/timeline/TimelineDataManager.java
* hadoop-yarn-project/CHANGES.txt
*
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice/src/test/java/org/apache/hadoop/yarn/server/timeline/webapp/TestTimelineWebServices.java
> Remove the logic to inject entity owner as the primary filter
> -------------------------------------------------------------
>
> Key: YARN-2818
> URL: https://issues.apache.org/jira/browse/YARN-2818
> Project: Hadoop YARN
> Issue Type: Bug
> Components: timelineserver
> Reporter: Zhijie Shen
> Assignee: Zhijie Shen
> Priority: Critical
> Fix For: 2.6.0
>
> Attachments: YARN-2818.1.patch, YARN-2818.2.patch
>
>
> In 2.5, we inject owner info as a primary filter to support entity-level
> acls. Since 2.6, we have a different acls solution (YARN-2102). Therefore,
> there's no need to inject owner info. There're two motivations:
> 1. For leveldb timeline store, the primary filter is expensive. When we have
> a primary filter, we need to make a complete copy of the entity on the logic
> index table.
> 2. Owner info is incomplete. Say we want to put E1 (owner = "tester",
> relatedEntity = "E2"). If E2 doesn't exist before, leveldb timeline store
> will create an empty E2 without owner info (at the db point of view, it
> doesn't know owner is a "special" primary filter).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
