steveloughran commented on PR #968:
URL: https://github.com/apache/parquet-mr/pull/968#issuecomment-1136465506

   > At this point the bottlenecks in parquet begin to move towards 
decompression and decoding but IO remains the slowest link in the chain.
   
   Latency is the killer; in an HTTP request you want read enough but not 
discard data or break an http connection if the client suddenly does a seek() 
or readFully() somewhere else. file listings, existence checks etc.
   
   > One thing we get with my PR is that the ParquetFileReader had assumptions 
built in that all data must be read before downstream can proceed. Some of my 
changes are related to removing these assumptions and ensuring that downstream 
processing does not block until an entire column is read so we get efficient 
pipelining.
   
   That'd be great. now, if you could also handle requesting different columns 
in parallel and processing them out of order.
   
   > What does the 128 MB block mean? Is this the amount prefetched for a 
stream? The read API does not block until the entire block is filled, I presume.
   
   this was the abfs client set to do four GET requests of 128MB each. this 
would be awful for columns stores where smaller ranges are often 
requested/processed before another seek is made, but quite often parquet does 
do more back to back reads than just one read/readFully request
   
   > With my PR, parquet IO is reading 8MB at a time (default) and downstream 
is processing 1MB at a time (default) and several such streams (one per column) 
are in progress at the same time. Hopefully, this read pattern would work with 
the prefetch.
   
   be good to think about vectored IO.
   
   and yes, updating parquet dependencies would be good, hadoop 3.3.0 should be 
the baseline.
   
   just sketched out my thoughts on this. I've played with some of this in my 
own branch. I think the next step would be for me to look at the benchmark code 
to make it targetable elsewhere.
   
   
   
https://docs.google.com/document/d/1y9oOSYbI6fFt547zcQJ0BD8VgvJWdyHBveaiCHzk79k/
   


-- 
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-unsubscr...@parquet.apache.org

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

Reply via email to