wujinhu commented on PR #6904:
URL: https://github.com/apache/hadoop/pull/6904#issuecomment-2230391625

   > Thanks. Looks good overall. I noticed this is mostly a lift from the S3A 
code. I am wondering if I have to re-write this, it would be good to create a 
VectoredInputStream which takes the actual DataInputStream as input and then 
all the Object stores like abfs, s3 and allyun extending this 
VectoredInputStream. Not really sure if this is feasible and will work.
   
   @mukund-thakur Thanks for your comments.  I think your idea is good, and we 
can open another issue to discuss 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 go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to