Thanks, Navina. Those two classes have been there for a while. So I think this is not really related to JobCoordinator/CoordinatorStream. Will just create a separate ticket for this. Of course, we can finish it within SAMZA-614.
Cheers, Fang, Yan yanfang...@gmail.com On Mon, Jun 15, 2015 at 11:39 AM, Navina Ramesh < nram...@linkedin.com.invalid> wrote: > +1 for adding to the documentation. I didnĀ¹t know about this difference. > This information is perhaps part of the JobCoordinator/CoordinatorStream > documentation - SAMZA-614? > > On 6/15/15, 11:27 AM, "Yan Fang" <yanfang...@gmail.com> wrote: > > >All right. Sorry for bothering. I figured it > >out. ApplicationMasterRestServlet is for the RPC and > >ApplicationMasterWebServlet is for tracking. The AM dashboard is from the > >tracking url, not he RPC url. Maybe we should mention the RPC url in the > >doc as well, since not many people realize this. > > > >Fang, Yan > >yanfang...@gmail.com > > > >On Fri, Jun 12, 2015 at 5:09 PM, Yan Fang <yanfang...@gmail.com> wrote: > > > >> Hi guys, > >> > >> I am looking at the code. Do we have any special reason that we want to > >> keep two web servlets in the AM? One is for the Rest, the other one is > >>for > >> the AM dashboard. Can we combine them together? > >> > >> This can reduce some duplicated code. Because it's good to have the AM > >> dashboard also show the information from the coordinatorStream, which > >> actually has been done in the Rest servlet. I think this also reduces > >>the > >> resource usage, though the reduction maybe not very significant. > >> > >> Cheers, > >> > >> Fang, Yan > >> yanfang...@gmail.com > >> > >