Li Lu commented on YARN-4224:

I scanned through the patch. The patch itself looks fine, but I would like to 
check with the broader community about the patterns proposed in this JIRA. 
Since this is blocking the ongoing web UI work, we really want to have a 
relatively stable interface on this before we proceed changing the UI side. 

IIUC we're putting the mandatory parameters in a hierarchical order in the URL, 
and adding optional parameters as query parameters. This approach looks fine 
with me. For naming conventions, we really want to be consistent with the rest 
part of the codebase. [~wangda] any suggestions/comments here, given this is 
quite related with the next-gen UI? Thanks! 

> Change the ATSv2 reader side REST interface to conform to current REST APIs' 
> in YARN
> ------------------------------------------------------------------------------------
>                 Key: YARN-4224
>                 URL: https://issues.apache.org/jira/browse/YARN-4224
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: timelineserver
>    Affects Versions: YARN-2928
>            Reporter: Varun Saxena
>            Assignee: Varun Saxena
>         Attachments: YARN-4224-YARN-2928.01.patch

This message was sent by Atlassian JIRA

Reply via email to