[GitHub] [hadoop] PrabhuJoseph commented on pull request #4486: YARN-10320.Replace FSDataInputStream#read with readFully in Log Aggregation

PrabhuJoseph commented on PR #4486: URL: https://github.com/apache/hadoop/pull/4486#issuecomment-1164625426 Latest Patch looks good, +1. Will commit it. -- 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

[GitHub] [hadoop] PrabhuJoseph commented on pull request #4486: YARN-10320.Replace FSDataInputStream#read with readFully in Log Aggregation

PrabhuJoseph commented on PR #4486: URL: https://github.com/apache/hadoop/pull/4486#issuecomment-1164027860 Thanks @ashutoshcipher for the patch. There is one more reference to read() at 804. Can you validate that as well. ` int actualLength = checksumFileInputStream.read(b);` --