impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Tim Armstrong <tarmstr...@cloudera.com>
Subject Re: Where to put shared, test-only BE code?
Date Wed, 14 Sep 2016 21:14:00 GMT
I think under either scheduler/ or testutil/ makes sense to me. I've seen
both - e.g. test-env.cc in runtime, or the various files in testutil.
Putting it under testutil/ seems a little cleaner to me.

I think we should just stick to linking each file into the main library for
its subdirectory, i.e. Scheduler or TestUtil. If the test utility code
isn't referenced from actual Impalad code it generally shouldn't be linked
into the final impalad binary.

On Wed, Sep 14, 2016 at 1:13 PM, Lars Volker <lv@cloudera.com> wrote:

> Hi all,
>
> I'm working on benchmarks for the SimpleScheduler and want to reuse the
> helper classes in simple-scheduler-test.cc for this. To do so, I'm going to
> factor them into their own files, which could be called
> simple-scheduler-test-util.{h,cc}.
>
> Where should those files live? And into which library should they be linked
> in the cmake configs?
>
> Thanks for the help, Lars
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message