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

Sumit Gupta commented on KNOX-719:
----------------------------------

Thanks [~jeffreyr97], The new patch seems to be the same as the old one. I'll 
move around the classes manually for now, but before I do a deep dive into the 
review I have a few high level observations:

1. I see some new classes that are marked as @Deprecated. What is the purpose 
of these?
2. Does this also require some changes to the service.xml file so that the 
correct classes are used? 
3. The main pom.xml needs to change to be able to add in the new module.

> Knox support for Yarn Resource Manager HA
> -----------------------------------------
>
>                 Key: KNOX-719
>                 URL: https://issues.apache.org/jira/browse/KNOX-719
>             Project: Apache Knox
>          Issue Type: Bug
>    Affects Versions: 0.6.0, 0.7.0, 0.8.0, 0.9.0, 0.10.0, 0.11.0
>            Reporter: Jeffrey E  Rodriguez
>            Assignee: Jeffrey E  Rodriguez
>             Fix For: 0.12.0
>
>         Attachments: KNOX-719-1.patch
>
>
> This would support both REST/UI YARN Resource Manager HA. Based on other HA 
> providers added in Knox.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to