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

Sean Mackrory edited comment on HADOOP-15688 at 8/23/18 1:36 AM:
-----------------------------------------------------------------

Thanks, Thomas. +1 on the patch. I didn't think to look in places other than 
the obvious open() path. I haven't been tracking specific instances of timeout 
tests, as there have been quite a lot of them when it does happen, but I'll 
start tracking that more. I'm also suspicious that my ISP might be partly to 
blame as I've been having some performance problems not just with Azure 
networks.


was (Author: mackrorysd):
Thanks, Thomas. +1 on the patch. I didn't think to look in places other than 
the obvious open() path. I haven't been tracking specific instances of timeout 
tests, as there have been quite a lot of them when it does happen. I'm also 
suspicious that my ISP might be partly to blame as I've been having some 
performance problems not just with Azure networks.

> ABFS: InputStream wrapped in FSDataInputStream twice
> ----------------------------------------------------
>
>                 Key: HADOOP-15688
>                 URL: https://issues.apache.org/jira/browse/HADOOP-15688
>             Project: Hadoop Common
>          Issue Type: Sub-task
>            Reporter: Sean Mackrory
>            Assignee: Sean Mackrory
>            Priority: Major
>         Attachments: HADOOP-15688-HADOOP-15407-002.patch, 
> HADOOP-15688.001.patch
>
>
> I can't read Parquet files from ABFS. It has 2 different implementations to 
> read seekable streams, and it'll use the one that uses ByteBuffer reads if it 
> can. It currently decides to use the ByteBuffer read implementation because 
> the FSDataInputStream it gets back wraps another FSDataInputStream, which 
> implements ByteBufferReadable.
> That's not the most robust way to check that ByteBufferReads are supported by 
> the ultimately underlying InputStream, but it's unnecessary and probably a 
> mistake to double-wrap the InputStream, so let's not.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to