Hean-Chhinling commented on PR #1178: URL: https://github.com/apache/knox/pull/1178#issuecomment-4109724240
> @Hean-Chhinling the rules look great, there are a few changes that I suggest you do and this should be ready to go. Thank you for your contribution to Knox. Thanks @moresandeep for the review! Regarding the dispatch class. I have created another class `gateway-service-rm/src/main/java/org/apache/knox/gateway/rm/dispatch/RMSchedulerUIHaDispatch.java` to handle YARN HA for SchedulerUI as follows the `RMUI2HaDispatch.java` and use `org.apache.knox.gateway.dispatch.ConfigurableDispatch`. Please kindly review on give your thoughts. I am in the process of spinning up a cluster and test in YARN HA. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
