Ying

Results 62 comments of Ying

> @yingsu00 Are there any plans to support Parquet v2 encoding? Yes I do have the plan to support v2 encodings, but it will be second half. Do you need...

@yzhang1991 I think the 2.x version refers to the DataPage version, and does not necessarily mean the encodings of the data are all V2. The writer could just be encoding...

@8dukongjian I took the liberty to edit the table and added a "seq" column to help us quickly identify the issues Also created [Parquet reading failed to decompress LZO files](https://github.com/facebookincubator/velox/issues/9618#top)...

> > Yes I do have the plan to support v2 encodings, but it will be second half. Do you need it urgently? > > @yingsu00 Yes, we need to...

@qqibrow @8dukongjian I will take a look at bug no. 9

@qqibrow We need a BooleanRleBpDataDecoder. There are a number of other types/decoders need to be added. I'll create a separate issue for it.

> Having partition key in file is a wrong thing to starts with. Where and when is the error introduced and how easy is it to fix it on the...

> We should have comment in code that we are doing this for unfixable bug in Iceberg This is not a bug of Iceberg but by design. Here is the...

@Yuhta I checked my code and see comments were already added: ``` // The partition key columns are written in the data file for // IcebergTables, so we need to...

> Curious is there a use case in Iceberg that we read only the data file but not the manifest? If no such case then the partition column in data...