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

Da Zhou commented on HADOOP-15778:
----------------------------------

one minor thing, instead of using literal string "bytes=", can we create a 
const string for it?
Overall looks good to me.

> ABFS: Fix client side throttling for read
> -----------------------------------------
>
>                 Key: HADOOP-15778
>                 URL: https://issues.apache.org/jira/browse/HADOOP-15778
>             Project: Hadoop Common
>          Issue Type: Sub-task
>            Reporter: Sneha Varma
>            Assignee: Sneha Varma
>            Priority: Minor
>         Attachments: HADOOP-15778-HADOOP-15407-001.patch
>
>
> 1. The content length for ReadFile in updateMetrics of 
> AbfsClientThrottlingIntercept is incorrect for cases when the request fails.
> It is currently equal to the number of bytes that are read whereas it should 
> be equal to the number of bytes requested.
> 2. sendingRequest of AbfsClientThrottlingIntercept at AbfsRestOperation 
> should be called irrespective of  whether the request has body.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to