You are viewing a plain text version of this content. The canonical link for it is here.
Posted to yarn-issues@hadoop.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2016/04/04 14:56:25 UTC

[jira] [Commented] (YARN-4746) yarn web services should convert parse failures of appId, appAttemptId and containerId to 400

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

Hudson commented on YARN-4746:
------------------------------

FAILURE: Integrated in Hadoop-trunk-Commit #9548 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/9548/])
YARN-4746. yarn web services should convert parse failures of appId, (naganarasimha_gr: rev 5092c94195a63bd2c3e36d5a74b4c061cea1b847)
* hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java
* hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/webapp/TestNMWebServicesApps.java
* hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/webapp/util/WebAppUtils.java
* hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/ConverterUtils.java
* hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java
* hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/java/org/apache/hadoop/yarn/server/webapp/WebServices.java
* hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesApps.java
* hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/webapp/NMWebServices.java


> yarn web services should convert parse failures of appId, appAttemptId and containerId to 400
> ---------------------------------------------------------------------------------------------
>
>                 Key: YARN-4746
>                 URL: https://issues.apache.org/jira/browse/YARN-4746
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: webapp
>    Affects Versions: 2.8.0
>            Reporter: Steve Loughran
>            Assignee: Bibin A Chundatt
>            Priority: Minor
>             Fix For: 2.8.0
>
>         Attachments: 0001-YARN-4746.patch, 0002-YARN-4746.patch, 0003-YARN-4746.patch, 0003-YARN-4746.patch, 0004-YARN-4746.patch, 0005-YARN-4746.patch
>
>
> I'm seeing somewhere in the WS API tests of mine an error with exception conversion of  a bad app ID sent in as an argument to a GET. I know it's in ATS, but a scan of the core RM web services implies a same problem
> {{WebServices.parseApplicationId()}} uses {{ConverterUtils.toApplicationId}} to convert an argument; this throws IllegalArgumentException, which is then handled somewhere by jetty as a 500 error.
> In fact, it's a bad argument, which should be handled by returning a 400. This can be done by catching the raised argument and explicitly converting it



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