Ryan Blue

Switching to use more Guava classes is probably not a good idea. @rizaon, do you have use cases where this has helped? If so, what were they? This adds quite...

@rizaon, caching in the FileIO layer would be much more general. You could do things like detect that the file size is less than some threshold and cache it in...

@rizaon, there are a couple of PRs that should help you with this. #4608 adds `IOUtil.readFully` and `IOUtil.readRemaining` that you can use to fill buffers from an input stream. I've...

@rizaon, the other PRs are in so you should be able to take advantage of `IOUtil.readFully` and the `ByteBufferInputStream` implementations. I hope that helps!

@findepi, @alexjo2144, FYI. This could be an option for Trino performance.

This looks great to me, other than a minor serialization convention (prefer arrays to lists for Kryo) and adding a public API method that I don't think we need. Thanks,...

Thanks, @nastra! Looks great. I merged this and we can follow up on #5773.

Thanks, @nastra! Looks great.

The new properties should be merged, not replaced. There are many properties that should live with the table across versions. For instance, the compression codec is based on the data...

Thanks, @findepi!