+1 Xabriel J Collazo Mojica | Senior Software Engineer | Adobe | xcoll...@adobe.com
From: Ted Gooch <tgo...@netflix.com.INVALID> Reply-To: "dev@iceberg.apache.org" <dev@iceberg.apache.org> Date: Tuesday, March 5, 2019 at 4:21 PM To: "dev@iceberg.apache.org" <dev@iceberg.apache.org> Cc: Ryan Blue <b...@apache.org> Subject: Re: [VOTE] Add the python implementation +1 On Tue, Mar 5, 2019 at 4:19 PM Ryan Blue <b...@apache.org<mailto:b...@apache.org>> wrote: +1 On Tue, Mar 5, 2019 at 4:18 PM Ryan Blue <b...@apache.org<mailto:b...@apache.org>> wrote: Hi everyone, I'd like to propose accepting the current Python PR, #54<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fincubator-iceberg%2Fpull%2F54&data=02%7C01%7Cxcollazo%40adobe.com%7C7ee0ecb319a24e73837808d6a1c9bbdd%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636874285150185399&sdata=pMSZpQ725aUC1TDd7P322i3KbsJYa6g8JdVkL0xtu8A%3D&reserved=0>. It has been through a round of reviews and while there is certainly more to do, it is going to be much easier to improve the implementation with smaller pull requests than to keep updating one giant PR. As noted in the discussion thread, this will not be included in the initial Java release. Also, the code would live in the existing repository where we already have visibility, until we have a reason to move it elsewhere. Please vote in the next 72 hours: [ ] +1: Commit the current Python PR implementation [ ] +0: . . . [ ] -1: Do not add the current implementation because . . . Thanks! rb -- Ryan Blue -- Ryan Blue
