Re: [C++] Control flow and scheduling in C++ Engine operators / exec nodes

2022-05-02 Thread Weston Pace
Thanks for investigating and looking through this. Your understanding of how things work is pretty much spot on. In addition, I think the points you are making are valid. Our ExecNode/ExecPlan interfaces are extremely bare bones and similar nodes have had to reimplement the same solutions (e.g.

Re: [VOTE][RUST] Release Apache Arrow Rust 13.0.0 RC1

2022-05-02 Thread Jiayu Liu
+1 (non-binding) Verified on M1 macos Monterey by running the verification script On Tue, May 3, 2022 at 2:02 AM Andy Grove wrote: > +1 (binding). > > Verified on Ubuntu 20.04.3 LTS (AMD) and macOS 12.3.1 (M1) > > On Mon, May 2, 2022 at 11:49 AM Chao Sun wrote: > > > +1 (non-binding), verified

[C++] Control flow and scheduling in C++ Engine operators / exec nodes

2022-05-02 Thread Wes McKinney
hi all, I've been catching up on the C++ execution engine codebase after a fairly long development hiatus. I have several questions / comments about the current design of the ExecNode and their implementations (currently: source / scan, filter, project, union, aggregate, sink, hash join). My cur

[VOTE] Release Apache Arrow 8.0.0 - RC2

2022-05-02 Thread Krisztián Szűcs
Hi, I would like to propose the following release candidate (RC2) of Apache Arrow version 8.0.0. This is a release consisting of 604 resolved JIRA issues[1]. This release candidate is based on commit: 77d35493c3878542613340a981eb5fcbddbcc24e [2] The source release rc2 is hosted at [3]. The binar

[DISC] (Java) Add Windows binaries to Maven packages

2022-05-02 Thread Larry White
Hi all, I would like to request your feedback on incorporating Windows binaries in those Maven packages that have native Arrow dependencies, while drawing your attention to the likely impact on jar size. Five of the 23 arrow packages on Maven Central have native dependencies. Four of those five

Re: [VOTE][RUST] Release Apache Arrow Rust 13.0.0 RC1

2022-05-02 Thread Andy Grove
+1 (binding). Verified on Ubuntu 20.04.3 LTS (AMD) and macOS 12.3.1 (M1) On Mon, May 2, 2022 at 11:49 AM Chao Sun wrote: > +1 (non-binding), verified on Intel Mac. Thanks Andrew! > > Chao > > On Fri, Apr 29, 2022 at 5:20 PM L. C. Hsieh wrote: > > > > +1 (non-binding), verified on Intel Mac. >

Re: [VOTE][RUST] Release Apache Arrow Rust 13.0.0 RC1

2022-05-02 Thread Chao Sun
+1 (non-binding), verified on Intel Mac. Thanks Andrew! Chao On Fri, Apr 29, 2022 at 5:20 PM L. C. Hsieh wrote: > > +1 (non-binding), verified on Intel Mac. > > Thanks Andrew! > > > Liang-Chi > > On Fri, Apr 29, 2022 at 3:42 PM Yang hao <1371656737...@gmail.com> wrote: > > > > +1 (non-binding) >

Re: [VOTE] Release Apache Arrow 8.0.0 - RC1

2022-05-02 Thread Krisztián Szűcs
Thanks for verifying and fixing the problem! I'm going to cut another RC including the patch. On Mon, May 2, 2022 at 2:44 PM David Li wrote: > > -1 for me due to the segfault in testing wheels. Weston and I were able to > reproduce and Weston filed a fix: https://github.com/apache/arrow/pull/1

Re: [VOTE] Release Apache Arrow 8.0.0 - RC1

2022-05-02 Thread David Li
-1 for me due to the segfault in testing wheels. Weston and I were able to reproduce and Weston filed a fix: https://github.com/apache/arrow/pull/13036 On Sat, Apr 30, 2022, at 21:39, Sutou Kouhei wrote: > +1 > > I ran the followings on Debian GNU/Linux sid: > > * TEST_DEFAULT=0 \ > TEST_