Re: [VOTE] Release Apache PyIceberg 0.2.1

2023-01-02 Thread Daniel Weeks
+1 (binding) Verified: - sigs/sums/license/tests - removal of hard dependency on pyarrow I don't believe the following should block this release unless they were introduced in this patch version, but I ran into the following: - pyparsing was not installed during 'make install' and tests failed

Re: [VOTE] Release Apache PyIceberg 0.2.1

2023-01-02 Thread Ryan Blue
+1 (binding) I checked licenses, checksum, and signature. Tests are passing, although in a new environment I get the error about not being able to load pyparsing. It would be nice to fix that, but since it isn't a regression, I think this release should not be blocked by it. Ryan On Mon, Jan 2,

Re: Iceberg Materialized View Spec

2023-01-02 Thread Ryan Blue
I just left a long comment on the issue. Thanks for picking this up, Jan! My main comments echoed what Walaa was talking about here. I also offered a different solution, which is to keep the MV metadata in snapshot properties in the table. That makes things smaller and I think it also addresses so

Re: [VOTE] Release Apache PyIceberg 0.2.1

2023-01-02 Thread Driesprong, Fokko
Hey everyone! First of all, happy new year! Best wishes for 2023 to you and your family. Just a gentle reminder to see if can release this RC to fix some critical bugs in 0.2.0. Thanks! Fokko Driesprong Op di 27 dec. 2022 om 09:17 schreef Driesprong, Fokko : > Hi Everyone, > > > First of all,