saxenapranav commented on code in PR #6699: URL: https://github.com/apache/hadoop/pull/6699#discussion_r1570058393
########## hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAbfsNetworkStatistics.java: ########## @@ -231,7 +237,17 @@ public void testAbfsHttpResponseStatistics() throws IOException { // 1 read request = 1 connection and 1 get response expectedConnectionsMade++; expectedGetResponses++; - expectedBytesReceived += bytesWrittenToFile; + if (!getConfiguration().getHeadOptimizationForInputStream()) { + expectedBytesReceived += bytesWrittenToFile; + } else { + /* + * With head optimization enabled, the abfsInputStream is not aware + * of the contentLength and hence, it would only read data for which the range + * is provided. With the first remote call done, the inputStream will get + * aware of the contentLength and would be able to use it for further reads. + */ + expectedBytesReceived += 1; Review Comment: With head optimization enabled and read footer optimization enabled, the abfsInputStream is not aware of the contentLength and hence, it would only read data for which the range is provided. With the first remote call done, the inputStream will get aware of the contentLength and would be able to use it for further reads. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org