Hi, Yes, I'll merge the fix and then remove cpp/src/arrow/dbi/hiveserver2.
Thanks, -- kou In <CAOCv4hgufcjaLeX55aFYRx=ielmnwwrg675fu9kcxsqlhqi...@mail.gmail.com> "Re: [C++] Can we remove cpp/src/arrow/dbi/hiveserver2?" on Mon, 6 Jun 2022 13:25:41 -0400, Neal Richardson <neal.p.richard...@gmail.com> wrote: > If you have fixes already for building it, perhaps merge those and then > remove it? That way, if we decided that we wanted to restore it (not that I > expect we would), we would be reverting to something that at least built > successfully. > > Neal > > > On Mon, Jun 6, 2022 at 1:04 PM Micah Kornfield <emkornfi...@gmail.com> > wrote: > >> +1 >> >> On Mon, Jun 6, 2022 at 8:58 AM Antoine Pitrou <anto...@python.org> wrote: >> >> > >> > +1 for removing it. >> > >> > >> > On Fri, 03 Jun 2022 08:32:35 +0900 (JST) >> > Sutou Kouhei <k...@clear-code.com> wrote: >> > > Hi, >> > > >> > > We have Hive adapter in cpp/src/arrow/dbi/hiveserver2 but >> > > it's not maintained. Can we remove this? >> > > >> > > Reasons: >> > > >> > > 1. I got build errors when I build it on master by >> > > -DARROW_HIVESERVER2=ON. See >> > > https://github.com/apache/arrow/pull/13292 for a fix of the >> > > build errors. >> > > >> > > We don't receive any bug report of it. It seems that >> > > nobody uses our Hive adapter. >> > > >> > > 2. We don't have a CI job that enables our Hive adapter. >> > > >> > > 3. Apache Arrow Flight SQL or Apache Arrow Database >> > > Connectivity may implement a feature to connect to Hive >> > > in future. >> > > >> > > >> > > Thanks, >> > >> > >> > >> > >>
