Rok Mihevc comments

Results 159 comments of


                                            Rok Mihevc

GH-15058: [C++][Python] Native support for UUID

Thanks for the review @pitrou! I've addressed all comments and waiting to see if CI shakes out any problems.

GH-15058: [C++][Python] Native support for UUID

Rebased. I'm not sure what the `value_type` method missing error when pickling is about. Checking.

GH-15058: [C++][Python] Native support for UUID

I think python should be ok now.

GH-15058: [C++][Python] Native support for UUID

Ping @jorisvandenbossche

GH-15058: [C++][Python] Native support for UUID

Thanks everyone!

GH-15058: [C++][Python] Native support for UUID

@jorisvandenbossche that sounds convenient! I've opened https://github.com/apache/arrow/issues/43855 and will get around to it if someone else doesn't sooner :).

GH-41719: [C++][Parquet] Cannot read encrypted parquet datasets via _metadata file

@AudriusButkevicius could you check if the read metadata contains expected properties?

GH-41719: [C++][Parquet] Cannot read encrypted parquet datasets via _metadata file

> Seems that dataset.get_fragments() doesn't return anything. I think c++ logic as-is doesn't store row groups, let me take a look and get back.

GH-41719: [C++][Parquet] Cannot read encrypted parquet datasets via _metadata file

> I don't think the design of parquet encryption has included the `_metadata` summary file because it may point to several different encrypted parquet files. Why would this be an...

GH-41719: [C++][Parquet] Cannot read encrypted parquet datasets via _metadata file

But the files can have the same key and decryption would error if not? That sounds ok to me.