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

ASF GitHub Bot commented on APEXCORE-201:
-----------------------------------------

Github user davidyan74 commented on a diff in the pull request:

    https://github.com/apache/incubator-apex-core/pull/194#discussion_r50581793
  
    --- Diff: 
engine/src/main/java/com/datatorrent/stram/StreamingContainerManager.java ---
    @@ -1402,6 +1313,7 @@ private void processOperatorDeployStatus(final 
PTOperator oper, OperatorHeartbea
             if (ds != null) {
               // operator was removed and needs to be undeployed from container
               sca.undeployOpers.add(oper.getId());
    +          slowestUpstreamOp.remove(oper.getId());
    --- End diff --
    
    @gauravgopi123  thanks for catching that. fixed!


> Reported latency is wrong when a downstream operator is behind more than 1000 
> windows
> -------------------------------------------------------------------------------------
>
>                 Key: APEXCORE-201
>                 URL: https://issues.apache.org/jira/browse/APEXCORE-201
>             Project: Apache Apex Core
>          Issue Type: Bug
>            Reporter: David Yan
>            Assignee: David Yan
>
> We should probably estimate this by reporting the latency using the number of 
> windows behind when that happens.  Right now it reports a stale latency.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to