I have been working with other committers to have the Gandiva PRs merged. What you see is the backlog of PRs that need to be merged after the review by Projjal.
On Mon, May 24, 2021 at 11:21 PM Will Jones <will.jones...@gmail.com> wrote: > Hello, > > Following up on this. Thanks to Projjal for reviewing so many PRs! One > issue is it doesn't look like Projjal is a committer and thus those PRs are > not shown as mergeable. > > Here are some PRs that looks like they should be ready to merge: > > > - ARROW-11342 <https://github.com/apache/arrow/pull/9288> > - ARROW-11341 <https://github.com/apache/arrow/pull/9289> > - ARROW-11960 <https://github.com/apache/arrow/pull/9700> > - ARROW-11549 <https://github.com/apache/arrow/pull/9453> > - ARROW-12205 <https://github.com/apache/arrow/pull/9890> > - ARROW-12378 <https://github.com/apache/arrow/pull/10023> > - ARROW-10155 <https://github.com/apache/arrow/pull/10155> > > > Thanks, > > Will Jones > > On Mon, Mar 29, 2021 at 1:31 AM Kirill Lykov <lykov.kir...@gmail.com> > wrote: > > > Thank you Micah for the effort. > > I think [python][gandiva] was added by Anyscale. Earlier, I've managed to > > invite them for the commit review yet they are currently not active with > > it. > > But these particular PRs are more about documentation and fixing some > minor > > problems and do not require in-depth knowledge of python/gandiva part. > > So I think a quick check by gandiva people on the documentation part of > PR > > would be valuable. > > > > > > On Mon, Mar 29, 2021 at 7:51 AM Projjal Chanda <i...@pchanda.com> wrote: > > > > > Hi Micah, Will, > > > I will review all the open C++/Java Gandiva PRs. Regarding the > > > [python][gandiva] PRs, I can take a look but its better if someone > > familiar > > > with it also reviews it. > > > > > > Thanks, > > > Projjal > > > > > > > On 29-Mar-2021, at 10:24 AM, Will Jones <will.jones...@gmail.com> > > wrote: > > > > > > > > Hi Vivek, > > > > > > > > The 18 PRs have the label "gandiva" in GitHub. Link: > > > > > > > > > > https://github.com/apache/arrow/pulls?q=is%3Aopen+is%3Apr+label%3Agandiva > > > > > > > > Best, > > > > Will > > > > > > > > > > > > On Sun, Mar 28, 2021 at 9:38 PM Vivekanand Vellanki < > vi...@dremio.com> > > > > wrote: > > > > > > > >> Micah, Is there a link where I can look at all the open Gandiva > PRs? I > > > will > > > >> update the thread with the list of PRs we are reviewing. > > > >> > > > >> On Mon, Mar 29, 2021 at 9:22 AM Micah Kornfield < > > emkornfi...@gmail.com> > > > >> wrote: > > > >> > > > >>> Hi Vivek, > > > >>> Does this apply to only PRs your team is opening or all open > Gandiva > > > PRs? > > > >>> > > > >>> Thanks, > > > >>> Micah > > > >>> > > > >>> On Thu, Mar 18, 2021 at 10:49 PM Vivekanand Vellanki < > > vi...@dremio.com > > > > > > > >>> wrote: > > > >>> > > > >>>> Micah, > > > >>>> > > > >>>> My team is looking at the PRs. We are giving feedback. > > > >>>> > > > >>>> We are also in touch with the committers (Ravindra and Praveen) > and > > > will > > > >>>> get them merged. > > > >>>> > > > >>>> Thanks > > > >>>> Vivek > > > >>>> > > > >>>> On Fri, Mar 19, 2021 at 9:29 AM Micah Kornfield < > > > emkornfi...@gmail.com> > > > >>>> wrote: > > > >>>> > > > >>>>> Is anybody actively looking at PRs for Gandiva? There seems to > be > > > >> queue > > > >>>>> building 18 (or so open). The committers that seemed to be > active > > in > > > >> the > > > >>>>> past don't seem to be responding to pings through Github. > > > >>>>> > > > >>>>> Thanks, > > > >>>>> Micah > > > >>>>> > > > >>>> > > > >> > > > > > > > > > > -- > > Best regards, > > Kirill Lykov > > >
