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

Kihwal Lee commented on HDFS-10303:
-----------------------------------

- There is a duplicate check for the heartbeat sequence number. The inner one 
is no longer necessary.
- The existing log level may not be ideal. This is directly visible to users 
and unless something actually failed, it is better to log at {{INFO}}.
- I am concerned about the new test case becoming one of the flaky ones that 
fail when running on a busy build machine. We need to make sure it is reliable. 

> DataStreamer#ResponseProcessor calculate packet acknowledge duration wrongly.
> -----------------------------------------------------------------------------
>
>                 Key: HDFS-10303
>                 URL: https://issues.apache.org/jira/browse/HDFS-10303
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: hdfs-client
>    Affects Versions: 2.7.2
>            Reporter: Surendra Singh Lilhore
>            Assignee: Surendra Singh Lilhore
>         Attachments: HDFS-10303-001.patch
>
>
> Packets acknowledge duration should be calculated based on the packet send 
> time.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to