samza-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Yan Fang <yanfang...@gmail.com>
Subject Re: Combine two web servlets in the AM?
Date Mon, 15 Jun 2015 18:57:38 GMT
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
yanfang724@gmail.com

On Mon, Jun 15, 2015 at 11:39 AM, Navina Ramesh <
nramesh@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" <yanfang724@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
> >yanfang724@gmail.com
> >
> >On Fri, Jun 12, 2015 at 5:09 PM, Yan Fang <yanfang724@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
> >> yanfang724@gmail.com
> >>
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message