[GitHub] [incubator-uniffle] jerqi commented on issue #239: [Problem] RssUtils#transIndexDataToSegments should consider the length of the data file

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

jerqi commented on issue #239:
URL: https://github.com/apache/incubator-uniffle/issues/239#issuecomment-1255775387

   > org.apache.uniffle.storage.handler.impl.HdfsFileReader#read
   > 
   > ```
   >   public byte[] read(long offset, int length) {
   >     try {
   >       fsDataInputStream.seek(offset);
   >       byte[] buf = new byte[length];
   >       fsDataInputStream.readFully(buf);
   >       return buf;
   >     } catch (Exception e) {
   >       LOG.warn("Can't read data for path:" + path + " with offset["
   >           + offset + "], length[" + length + "]", e);
   >     }
   >     return new byte[0];
   >   }
   > ```
   
   So we should process EOFException in this case.


-- 
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: dev-unsubscribe@uniffle.apache.org

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