[Impala-ASF-CR] Add -build shared libs for default build for speed.

2016-11-08 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: Add -build_shared_libs for default build for speed. .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4970 To unsubscribe, visit

[Impala-ASF-CR] Add -build shared libs for default build for speed.

2016-11-08 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: Add -build_shared_libs for default build for speed. .. Add -build_shared_libs for default build for speed. This is already recommended by the wiki:

[Impala-ASF-CR] Add -build shared libs for default build for speed.

2016-11-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Add -build_shared_libs for default build for speed. .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4970 To unsubscribe, visit

[Impala-ASF-CR] Add -build shared libs for default build for speed.

2016-11-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Add -build_shared_libs for default build for speed. .. Patch Set 1: Good point. I was curious about what others did. It looks like MapR also ships a

[Impala-ASF-CR] Add -build shared libs for default build for speed.

2016-11-08 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: Add -build_shared_libs for default build for speed. .. Patch Set 1: > I'm fine with this change, but I remember Dan had some reservations > when this change was

[Impala-ASF-CR] Add -build shared libs for default build for speed.

2016-11-08 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add -build_shared_libs for default build for speed. .. Patch Set 1: > > I'm fine with this change, but I remember Dan had some > reservations > > when this change

[Impala-ASF-CR] Add -build shared libs for default build for speed.

2016-11-08 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add -build_shared_libs for default build for speed. .. Patch Set 1: > I'm ok with this as long as we have automated test coverage of this > script (in case the

[Impala-ASF-CR] Add -build shared libs for default build for speed.

2016-11-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Add -build_shared_libs for default build for speed. .. Patch Set 1: Code-Review+1 I'm ok with this as long as we have automated test coverage of this script (in

[Impala-ASF-CR] Add -build shared libs for default build for speed.

2016-11-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: Add -build_shared_libs for default build for speed. .. Patch Set 1: I'm fine with this change, but I remember Dan had some reservations when this change was

[Impala-ASF-CR] Add -build shared libs for default build for speed.

2016-11-06 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Add -build_shared_libs for default build for speed. .. Patch Set 1: Code-Review+1 I use this, too, and it works well. -- To view, visit

[Impala-ASF-CR] Add -build shared libs for default build for speed.

2016-11-05 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/4970 Change subject: Add -build_shared_libs for default build for speed. .. Add -build_shared_libs for default build for speed. This is already