samza-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Navina Ramesh <nram...@linkedin.com.INVALID>
Subject Re: Combine two web servlets in the AM?
Date Mon, 15 Jun 2015 18:39:34 GMT
+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
View raw message