hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hadoop QA (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-1732) Change types of related entities and primary filters in ATSEntity
Date Fri, 14 Feb 2014 05:15:19 GMT

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

Hadoop QA commented on YARN-1732:
---------------------------------

{color:red}-1 overall{color}.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12628944/YARN-1732.1.patch
  against trunk revision .

    {color:green}+1 @author{color}.  The patch does not contain any @author tags.

    {color:green}+1 tests included{color}.  The patch appears to include 1 new or modified
test files.

    {color:red}-1 javac{color:red}.  The patch appears to cause the build to fail.

Console output: https://builds.apache.org/job/PreCommit-YARN-Build/3102//console

This message is automatically generated.

> Change types of related entities and primary filters in ATSEntity
> -----------------------------------------------------------------
>
>                 Key: YARN-1732
>                 URL: https://issues.apache.org/jira/browse/YARN-1732
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Billie Rinaldi
>            Assignee: Billie Rinaldi
>         Attachments: YARN-1732.1.patch
>
>
> The current types Map<String, List<String>> relatedEntities and Map<String,
Object> primaryFilters have issues.  The List<String> value of the related entities
map could have multiple identical strings in it, which doesn't make sense. A more major issue
is that we cannot allow primary filter values to be overwritten, because otherwise we will
be unable to find those primary filter entries when we want to delete an entity (without doing
a nearly full scan).
> I propose changing related entities to Map<String, Set<String>> and primary
filters to Map<String, Set<Object>>.  The basic methods to add primary filters
and related entities are of the form add(key, value) and will not need to change.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Mime
View raw message