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

ASF GitHub Bot commented on HADOOP-18391:
-----------------------------------------

steveloughran commented on code in PR #4787:
URL: https://github.com/apache/hadoop/pull/4787#discussion_r957189351


##########
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/functional/Function4RaisingIOE.java:
##########
@@ -0,0 +1,8 @@
+package org.apache.hadoop.util.functional;
+
+import java.io.IOException;
+
+public interface Function4RaisingIOE<I1, I2, I3, I4> {

Review Comment:
   add a return type, R, as last arg, so it actually becomes a function.





> Improve VectoredReadUtils
> -------------------------
>
>                 Key: HADOOP-18391
>                 URL: https://issues.apache.org/jira/browse/HADOOP-18391
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs
>    Affects Versions: 3.3.9
>            Reporter: Steve Loughran
>            Assignee: Mukund Thakur
>            Priority: Major
>              Labels: pull-request-available
>
> harden the VectoredReadUtils methods for consistent and more robust use, 
> especially in those filesystems which don't have the api.
> VectoredReadUtils.readInDirectBuffer should allocate a max buffer size, .e.g 
> 4mb, then do repeated reads and copies; this ensures that you don't OOM with 
> many threads doing ranged requests. other libs do this.
> readVectored to call validateNonOverlappingAndReturnSortedRanges before 
> iterating
> this ensures the abfs/s3a requirements are always met, and that because 
> ranges will be read in order, prefetching by other clients will keep their 
> performance good.
> readVectored to add special handling for 0 byte ranges



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to