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

Jian He commented on YARN-7202:
-------------------------------

confused, isn't this code not part of this patch ? I did clean before build.
Can you try compile only this patch with the dependency removed? 
{code}
[ERROR] 
/home/eyang/test/hadoop/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services-api/src/test/java/org/apache/hadoop/yarn/service/ServiceClientTest.java:[29,45]
 cannot find symbol
[ERROR]   symbol:   class YarnSolrClientTest
{code}

bq.  The new code enables the logic to be handled more accurately and provide 
better status report to down stream program weather the failure is caused by a 
bad request, record not found, or undetermined server errors. The previous 
implementation does not allow certain error to be surfaced correctly.
Confused. I'm seeing the opposite. 
Before the patch, it directly returns the status code based on whatever failure 
conditions it is inside the method which is accurate.
After the patch, it only returns INTERNAL_SERVER_ERROR or BAD_REQUEST.  e.g. 
the NOT_FOUND  status code for stopService method is instead converted to the 
BAD_REQUEST status code.  Why is this converstion needed? 

> End-to-end UT for api-server
> ----------------------------
>
>                 Key: YARN-7202
>                 URL: https://issues.apache.org/jira/browse/YARN-7202
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Jian He
>            Assignee: Eric Yang
>         Attachments: YARN-7202.yarn-native-services.001.patch, 
> YARN-7202.yarn-native-services.002.patch, 
> YARN-7202.yarn-native-services.003.patch, 
> YARN-7202.yarn-native-services.004.patch, 
> YARN-7202.yarn-native-services.005.patch, 
> YARN-7202.yarn-native-services.006.patch, 
> YARN-7202.yarn-native-services.007.patch, 
> YARN-7202.yarn-native-services.008.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to