Ryan Blue

Thanks for doing this, @pradeepg26! We've occasionally had use cases where it would have been nice. My two main concerns are the unnecessary boolean controlling whether or not to estimate...

@pwais, this was replaced by #470 that includes updates for problems in this PR. Unfortunately, other committers decided they did not want to change internal APIs so it was not...

@MickDavies: I don't think you need to use thrift 0.7.0. I've built with 0.9.0 without problems, and newer versions (before 0.9.2 at least) should work.

There's a lot of behavior to consider for processing engines here, but the conclusion I think we can draw from all of this information for Parquet is that engines may...

Thanks @spena, I'll take a look at this today.

@isnotinvain, I think this was fixed by a later PR and can be closed.

Thanks for posting a patch @ptkool! Is this preventing Parquet from being used for some cases? I want to make sure the trade-off is worth including this. Right now, we...

@julienledem, the problem is that this allows names in Parquet files that will break some object models.

I'm still -1 on this because it is allowing names that we know will break object models. I wouldn't say that Thrift, Protobuf, and Avro are only used to import...

> size command is still expecting glob path. I feel it is helpful but let me know if you find it otherwise. Initially, I thought that these should work on...