[
https://issues.apache.org/jira/browse/YARN-5412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16102197#comment-16102197
]
Giovanni Matteo Fumarola commented on YARN-5412:
------------------------------------------------
Thanks Carlo for the feedback - uploaded v5.
1. I validated that the {{hadoop-yarn-server-router/pom.xml}} is minimal.
2. Good catch.
3. I updated the tests to be tighter.
4. I added them.
5. I implemented them.
6. It cannot happen since the Router calls that method.
7. For GET calls, you can receive 200 or null.
8. Same behavior of Yarn RM - return null.
9. Same behavior of Yarn RM.
10. Reduced for POST/PUT calls.
11. Router just forwards the calls to the Yarn RM. Wrong and null input is a
validate inside the RM. I added check for wrong http method.
However, thanks to {{RMWebServicesProtocol}} we added in YARN-6587 we filter
what we can get in input.
12. I tried but some params are static due to {{beforeClass}} method.
I addressed bunch of those.
> Create a proxy chain for ResourceManager REST API in the Router
> ---------------------------------------------------------------
>
> Key: YARN-5412
> URL: https://issues.apache.org/jira/browse/YARN-5412
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: nodemanager, resourcemanager
> Reporter: Subru Krishnan
> Assignee: Giovanni Matteo Fumarola
> Attachments: YARN-5412-YARN-2915.1.patch,
> YARN-5412-YARN-2915.2.patch, YARN-5412-YARN-2915.3.patch,
> YARN-5412-YARN-2915.4.patch, YARN-5412-YARN-2915.5.patch,
> YARN-5412-YARN-2915.proto.patch
>
>
> As detailed in the proposal in the umbrella JIRA, we are introducing a new
> component that routes client request to appropriate ResourceManager(s). This
> JIRA tracks the creation of a proxy for ResourceManager REST API in the
> Router. This provides a placeholder for:
> 1) throttling mis-behaving clients (YARN-1546)
> 3) mask the access to multiple RMs (YARN-3659)
> We are planning to follow the interceptor pattern like we did in YARN-2884 to
> generalize the approach and have only dynamically coupling for Federation.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
