Re: status-benchmark.cc compilation time

2017-02-23 Thread Henry Robinson
I think the main problem I want to avoid is paying the cost of linking, which is expensive for Impala as it often generates multi-hundred-MB binaries per benchmark or test. Building the benchmarks during GVO seems the best solution to that to me. On 23 February 2017 at 10:23, Todd Lipcon

Re: status-benchmark.cc compilation time

2017-02-23 Thread Todd Lipcon
One thing we've found useful in Kudu to prevent bitrot of benchmarks is to actually use gtest and gflags for the benchmark programs. We set some flag like --benchmark_num_rows or --benchmark_num_iterations with a default that's low enough to only run for a second or two, and run it as part of our

Re: status-benchmark.cc compilation time

2017-02-23 Thread Henry Robinson
Fair enough, always worth checking the easy things first :) On 23 February 2017 at 10:20, Zachary Amsden wrote: > Yes. If you take a look at the benchmark, you'll notice the JNI call to > initialize the frontend doesn't even have the right signature anymore. > That's one

Re: status-benchmark.cc compilation time

2017-02-23 Thread Zachary Amsden
Yes. If you take a look at the benchmark, you'll notice the JNI call to initialize the frontend doesn't even have the right signature anymore. That's one easy way to bitrot while still compiling. Even fixing that isn't enough to get it off the ground. - Zach On Tue, Feb 21, 2017 at 11:44 AM,

[Toolchain-CR] Update Kudu version to 2b0edbe

2017-02-23 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: Update Kudu version to 2b0edbe .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6110 To unsubscribe, visit

[Toolchain-CR] Update Kudu version to 2b0edbe

2017-02-23 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: Update Kudu version to 2b0edbe .. Update Kudu version to 2b0edbe Change-Id: I175133b1841449e311d637d10efc9d503612a3cf --- M buildall.sh 1 file

[Toolchain-CR] Update Kudu version to 2b0edbe

2017-02-23 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Update Kudu version to 2b0edbe .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6110 To unsubscribe, visit

New Impala committer: Thomas Marshall

2017-02-23 Thread Matthew Jacobs
The Podling Project Management Committee (PPMC) for Apache Impala (incubating) has invited Thomas Marshall to become a committer and we are pleased to announce that he has accepted. Congratulations and welcome, Thomas!

New Impala committer: Michael Brown

2017-02-23 Thread Jim Apple
The Podling Project Management Committee (PPMC) for Apache Impala (incubating) has invited Michael Brown to become a committer and we are pleased to announce that he has accepted. Congratulations and welcome, Michael!