[GitHub] [ozone] sodonnel commented on a diff in pull request #3764: HDDS-7235. EOFException occurs when executing TPC-DS using o3fs

[GitBox <gi...@apache.org>]

sodonnel commented on code in PR #3764:
URL: https://github.com/apache/ozone/pull/3764#discussion_r984727151


##########
hadoop-hdds/client/src/test/java/org/apache/hadoop/hdds/scm/storage/TestChunkInputStream.java:
##########
@@ -147,14 +145,6 @@ public void testPartialChunkRead() throws Exception {
   public void testSeek() throws Exception {
     seekAndVerify(0);
 
-    try {

Review Comment:
   We have just removed the EOF check on seek. Perhaps rather than removing it, we should ensure the EOF is still thrown at CHUNK_SIZE + 1?



-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org