You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2015/02/05 19:28:36 UTC

[jira] [Commented] (AMBARI-9495) Call to save configs of different services together should 400 return code not 500

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

Hudson commented on AMBARI-9495:
--------------------------------

SUCCESS: Integrated in Ambari-trunk-Commit #1696 (See [https://builds.apache.org/job/Ambari-trunk-Commit/1696/])
AMBARI-9495. Call to save configs of different services together should 400 return code not 500 (aonishuk) (aonishuk: http://git-wip-us.apache.org/repos/asf?p=ambari.git&a=commit&h=c52f70bcfdc207e7ea2dd4a5e8966855d631e61f)
* ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
* ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterTest.java
* ambari-server/src/test/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostTest.java
* ambari-server/src/main/java/org/apache/ambari/server/state/Cluster.java


> Call to save configs of different services together should 400 return code not 500
> ----------------------------------------------------------------------------------
>
>                 Key: AMBARI-9495
>                 URL: https://issues.apache.org/jira/browse/AMBARI-9495
>             Project: Ambari
>          Issue Type: Bug
>            Reporter: Andrew Onischuk
>            Assignee: Andrew Onischuk
>             Fix For: 2.0.0
>
>
> This AmbariException will need to be caught upstream and translated to an
> IllegalArgumentException (or thrown from here) which will result in the
> expected 400 Bad Request response back to the user. If the AmbariException is
> not caught, it will result in a 500 Server Error to the user which is not
> appropriate.



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