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-2936) YARNDelegationTokenIdentifier doesn't set proto.builder now
Date Thu, 18 Dec 2014 19:55:13 GMT

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

Hadoop QA commented on YARN-2936:
---------------------------------

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

    {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:green}+1 javac{color}.  The applied patch does not increase the total number of
javac compiler warnings.

    {color:green}+1 javadoc{color}.  There were no new javadoc warning messages.

    {color:green}+1 eclipse:eclipse{color}.  The patch built with eclipse:eclipse.

    {color:red}-1 findbugs{color}.  The patch appears to introduce 39 new Findbugs (version
2.0.3) warnings.

    {color:green}+1 release audit{color}.  The applied patch does not increase the total number
of release audit warnings.

    {color:red}-1 core tests{color}.  The patch failed these unit tests in hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager:

                  org.apache.hadoop.yarn.server.resourcemanager.TestWorkPreservingRMRestart
                  org.apache.hadoop.yarn.server.resourcemanager.TestRM

                                      The following test timeouts occurred in hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager:

org.apache.hadoop.yarn.server.resourcemanager.TestRMRestart

Test results: https://builds.apache.org/job/PreCommit-YARN-Build/6147//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-YARN-Build/6147//artifact/patchprocess/newPatchFindbugsWarningshadoop-yarn-server-resourcemanager.html
Findbugs warnings: https://builds.apache.org/job/PreCommit-YARN-Build/6147//artifact/patchprocess/newPatchFindbugsWarningshadoop-yarn-common.html
Console output: https://builds.apache.org/job/PreCommit-YARN-Build/6147//console

This message is automatically generated.

> YARNDelegationTokenIdentifier doesn't set proto.builder now
> -----------------------------------------------------------
>
>                 Key: YARN-2936
>                 URL: https://issues.apache.org/jira/browse/YARN-2936
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Zhijie Shen
>            Assignee: Varun Saxena
>         Attachments: YARN-2936.001.patch
>
>
> After YARN-2743, the setters are removed from YARNDelegationTokenIdentifier, such that
when constructing a object which extends YARNDelegationTokenIdentifier, proto.builder is not
set at all. Later on, when we call getProto() of it, we will just get an empty proto object.
> It seems to do no harm to the production code path, as we will always call getBytes()
before using proto to persist the DT in the state store, when we generating the password.
> I think the setter is removed to avoid duplicating setting the fields why getBytes()
is called. However, YARNDelegationTokenIdentifier doesn't work properly alone. YARNDelegationTokenIdentifier
is tightly coupled with the logic in secretManager. It's vulnerable if something is changed
at secretManager. For example, in the test case of YARN-2837, I spent time to figure out we
need to execute getBytes() first to make sure the testing DTs can be properly put into the
state store.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message