[Impala-CR](cdh5-trunk) IMPALA-3148. Fix selectivity computation for pushed Kudu predicates

2016-04-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3148. Fix selectivity computation for pushed Kudu predicates .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2628 To unsubsc

[Impala-CR](cdh5-trunk) IMPALA-3148. Fix selectivity computation for pushed Kudu predicates

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has posted comments on this change. Change subject: IMPALA-3148. Fix selectivity computation for pushed Kudu predicates .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/2628/2/testdata/workloads/fun

[Impala-CR](cdh5-trunk) IMPALA-3148. Fix selectivity computation for pushed Kudu predicates

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new patch set (#3). Change subject: IMPALA-3148. Fix selectivity computation for pushed Kudu predicates .. IMPALA-3148. Fix selectivity computation for pushed Kudu predicates This follows up on a TODO

[Impala-CR](cdh5-trunk) IMPALA-3269: Remove authz checks on default table location in CTAS queries

2016-04-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3269: Remove authz checks on default table location in CTAS queries .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/2664/1/fe/src/test/java/

[Impala-CR](cdh5-trunk) IMPALA-3148. Fix selectivity computation for pushed Kudu predicates

2016-04-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3148. Fix selectivity computation for pushed Kudu predicates .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/2628/2/testdata/workloads/funct

Last-minute April podling report

2016-04-05 Thread Henry Robinson
Hi all - Below is my draft report for Impala for April. I'm going to copy it straight away to the wiki (http://wiki.apache.org/incubator/April2016), so please make changes there as you see fit. Henry Impala Impala is a high-performance C++ and Java SQL query engine for data stored in Apache Had

[Impala-CR](cdh5-trunk) IMPALA-3148. Fix selectivity computation for pushed Kudu predicates

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has posted comments on this change. Change subject: IMPALA-3148. Fix selectivity computation for pushed Kudu predicates .. Patch Set 2: I added a few more tests and reverted the one Todd modified. I think is test

[Impala-CR](cdh5-trunk) IMPALA-3148. Fix selectivity computation for pushed Kudu predicates

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new patch set (#2). Change subject: IMPALA-3148. Fix selectivity computation for pushed Kudu predicates .. IMPALA-3148. Fix selectivity computation for pushed Kudu predicates This follows up on a TODO

[Impala-CR](cdh5-trunk) Updates for new Kudu toolchain layout and upgrade Kudu

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new patch set (#2). Change subject: Updates for new Kudu toolchain layout and upgrade Kudu .. Updates for new Kudu toolchain layout and upgrade Kudu The directory structure of the newer Kudu toolchain

[Impala-CR](cdh5-2.5.0_5.7.0) IMPALA-3236: Increase timeout for runtime filter tests

2016-04-05 Thread Henry Robinson (Code Review)
Henry Robinson has submitted this change and it was merged. Change subject: IMPALA-3236: Increase timeout for runtime filter tests .. IMPALA-3236: Increase timeout for runtime filter tests When running with ASAN enabled, runtim

[Impala-CR](cdh5-2.5.0_5.7.0) IMPALA-3236: Increase timeout for runtime filter tests

2016-04-05 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-3236: Increase timeout for runtime filter tests .. Patch Set 2: Code-Review+2 Verified+1 Just changed the JIRA number. Carrying on the +1 from patch 1, a

[Impala-CR](cdh5-2.5.0_5.7.0) IMPALA-3236: Increase timeout for runtime filter tests

2016-04-05 Thread Henry Robinson (Code Review)
Hello Michael Brown, Internal Jenkins, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2657 to look at the new patch set (#2). Change subject: IMPALA-3236: Increase timeout for runtime filter tests

[Impala-CR](cdh5-trunk) CDH-38939: Update hive-jdbc.version to corresponding cdh version

2016-04-05 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: CDH-38939: Update hive-jdbc.version to corresponding cdh version .. Patch Set 1: Is there an IMPALA- JIRA for this? -- To view, visit http://gerrit.cloudera.o

[Impala-CR](cdh5-trunk) IMPALA-3301: TestParquet::test_resolution_by_name fails on legacy join/agg nodes

2016-04-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3301: TestParquet::test_resolution_by_name fails on legacy join/agg nodes .. IMPALA-3301: TestParquet::test_resolution_by_name fails on legacy

[Toolchain-CR] Build Kudu from source

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has posted comments on this change. Change subject: Build Kudu from source .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/2679/3/functions.sh File functions.sh: Line 133: # Usage: header [ > Yo

[Toolchain-CR] Build Kudu from source

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new patch set (#6). Change subject: Build Kudu from source .. Build Kudu from source With this change any arbitrary Kudu hash can be built. Previously only a released version of Kudu could be used beca

[Toolchain-CR] Add TPC-H and TPC-DS

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new patch set (#4). Change subject: Add TPC-H and TPC-DS .. Add TPC-H and TPC-DS TPC-H and DS aren't for build purposes. They are provided here as a convenience. This provides the ability to generate d

[Toolchain-CR] Build Kudu from source

2016-04-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Build Kudu from source .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/2679/3/functions.sh File functions.sh: Line 133: # Usage: header [ You s

[Impala-CR](cdh5-trunk) Headers and reservation logic for daemon-wide buffer pool

2016-04-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#5). Change subject: Headers and reservation logic for daemon-wide buffer pool .. Headers and reservation logic for daemon-wide buffer pool This patch implements the core interface and

[Toolchain-CR] Add TPC-H and TPC-DS

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new patch set (#3). Change subject: Add TPC-H and TPC-DS .. Add TPC-H and TPC-DS TPC-H and DS aren't for build purposes. They are provided here as a convenience. This provides the ability to generate d

[Toolchain-CR] Build Kudu from source

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new patch set (#5). Change subject: Build Kudu from source .. Build Kudu from source With this change any arbitrary Kudu hash can be built. Previously only a released version of Kudu could be used beca

[Impala-CR](cdh5-2.5.0_5.7.0) IMPALA-3226: Increase timeout for runtime filter tests

2016-04-05 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-3226: Increase timeout for runtime filter tests .. Patch Set 1: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/2657/1//COMMIT_MSG Commit M

[Toolchain-CR] Build Kudu from source

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new patch set (#4). Change subject: Build Kudu from source .. Build Kudu from source With this change any arbitrary Kudu hash can be built. Previously only a released version of Kudu could be used beca

[Toolchain-CR] Add TPC-H and TPC-DS

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new patch set (#2). Change subject: Add TPC-H and TPC-DS .. Add TPC-H and TPC-DS TPC-H and DS aren't for build purposes. They are provided here as a convenience. This provides the ability to generate d

[Impala-CR](cdh5-trunk) WIP: IMPALA-3148. Fix selectivity computation for pushed Kudu predicates

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has posted comments on this change. Change subject: WIP: IMPALA-3148. Fix selectivity computation for pushed Kudu predicates .. Patch Set 1: I think this looks good. I'll update the patch with a few more planner te

[Impala-CR](cdh5-trunk) IMPALA-1878: Support INSERT and LOAD DATA on S3 and between filesystems

2016-04-05 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-1878: Support INSERT and LOAD DATA on S3 and between filesystems .. Patch Set 10: (12 comments) http://gerrit.cloudera.org:8080/#/c/2574/10/be/src/runti

[Impala-CR](cdh5-trunk) IMPALA-1878: Support INSERT and LOAD DATA on S3 and between filesystems

2016-04-05 Thread Sailesh Mukil (Code Review)
Hello Taras Bobrovytsky, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2574 to look at the new patch set (#11). Change subject: IMPALA-1878: Support INSERT and LOAD DATA on S3 and between filesystems ..

Board Report

2016-04-05 Thread John D. Ament
Dear Impala Podling, Just a reminder about your board report, which is due tomorrow. John

[Impala-CR](cdh5-trunk) IMPALA-3245: Fix crash with global filters when NUM_NODES=1

2016-04-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3245: Fix crash with global filters when NUM_NODES=1 .. Patch Set 1: Ahh I did not see IMPALA-3305. The current code is even semantically not right. We compu

[Impala-CR](cdh5-trunk) IMPALA-3245: Fix crash with global filters when NUM_NODES=1

2016-04-05 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-3245: Fix crash with global filters when NUM_NODES=1 .. Patch Set 1: That seems a little subtle. Perhaps it would be better to fix IMPALA-3305 directly

[Impala-CR](cdh5-trunk) IMPALA-3301: TestParquet::test_resolution_by_name fails on legacy join/agg nodes

2016-04-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3301: TestParquet::test_resolution_by_name fails on legacy join/agg nodes .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/27

[Impala-CR](cdh5-trunk) IMPALA-2548: Codegen Tuple::MaterializeExprs() and use in TopN node

2016-04-05 Thread Skye Wanderman-Milne (Code Review)
Skye Wanderman-Milne has posted comments on this change. Change subject: IMPALA-2548: Codegen Tuple::MaterializeExprs() and use in TopN node .. Patch Set 11: ping -- To view, visit http://gerrit.cloudera.org:8080/1901 To uns

[Impala-CR](cdh5-trunk) IMPALA-1878: Support INSERT and LOAD DATA on S3 and between filesystems

2016-04-05 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-1878: Support INSERT and LOAD DATA on S3 and between filesystems .. Patch Set 10: (11 comments) http://gerrit.cloudera.org:8080/#/c/2574/10/be/src/runt

[Impala-CR](cdh5-trunk) CDH-38939: Update hive-jdbc.version to corresponding cdh version

2016-04-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: CDH-38939: Update hive-jdbc.version to corresponding cdh version .. Patch Set 1: Code-Review+2 I'm doing a private build for Charlie. -- To view, visit http://gerr

[Impala-CR](cdh5-trunk) IMPALA-3245: Fix crash with global filters when NUM_NODES=1

2016-04-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3245: Fix crash with global filters when NUM_NODES=1 .. Patch Set 1: Maybe I'm misunderstanding, but could we not fix this problem by setting RuntimeFilter.h

[Impala-CR](cdh5-trunk) IMPALA-3301: TestParquet::test_resolution_by_name fails on legacy join/agg nodes

2016-04-05 Thread Skye Wanderman-Milne (Code Review)
Skye Wanderman-Milne has uploaded a new change for review. http://gerrit.cloudera.org:8080/2723 Change subject: IMPALA-3301: TestParquet::test_resolution_by_name fails on legacy join/agg nodes .. IMPALA-3301: TestParquet::tes

[Impala-CR](cdh5-trunk) CDH-38939: Update hive-jdbc.version to corresponding cdh version

2016-04-05 Thread Charlie Helin (Code Review)
Charlie Helin has uploaded a new change for review. http://gerrit.cloudera.org:8080/2722 Change subject: CDH-38939: Update hive-jdbc.version to corresponding cdh version .. CDH-38939: Update hive-jdbc.version to corresponding

[Impala-CR](cdh5-trunk) IMPALA-2645: Set mechUsing only on success

2016-04-05 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-2645: Set mechUsing only on success .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2687 To unsubscribe, visit http://ge

[Impala-CR](cdh5-trunk) IMPALA-2645: Set mechUsing only on success

2016-04-05 Thread Henry Robinson (Code Review)
Henry Robinson has submitted this change and it was merged. Change subject: IMPALA-2645: Set mechUsing only on success .. IMPALA-2645: Set mechUsing only on success Better to check success status from sasl_client_start() before

[Impala-CR](cdh5-trunk) Disable Kudu on SLES and Debian

2016-04-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: Disable Kudu on SLES and Debian .. Disable Kudu on SLES and Debian Kudu won't work on these platforms yet. Change-Id: I38bf6b803b3afb80aa1011522d21f

[Impala-CR](cdh5-trunk) IMPALA-3245: Fix crash with global filters when NUM_NODES=1

2016-04-05 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/2721 Change subject: IMPALA-3245: Fix crash with global filters when NUM_NODES=1 .. IMPALA-3245: Fix crash with global filters when NUM_NODES

[Impala-CR](cdh5-trunk) Updates for new Kudu toolchain layout and upgrade Kudu

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new change for review. http://gerrit.cloudera.org:8080/2720 Change subject: Updates for new Kudu toolchain layout and upgrade Kudu .. Updates for new Kudu toolchain layout and upgrade Kudu The direc

[Toolchain-CR] Fix LLVM linker bug

2016-04-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Fix LLVM linker bug .. Patch Set 3: Updated the patch to include a unit test, reflecting what I put up for review upstream. -- To view, visit http://gerrit.cl

[Toolchain-CR] Fix LLVM linker bug

2016-04-05 Thread Tim Armstrong (Code Review)
Hello Casey Ching, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2656 to look at the new patch set (#3). Change subject: Fix LLVM linker bug .. Fix LLVM linker bug The LLVM linker has a

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-05 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/2543/10/be/src/util/decompress.cc File be/src

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-05 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/2543/10/be/src/util/decompress.cc File be/src/u

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-05 Thread Juan Yu (Code Review)
Hello Skye Wanderman-Milne, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2543 to look at the new patch set (#11). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage ..

Re: maven error while compiling the front end code

2016-04-05 Thread Pradeep Nayak
This is what I saw in IMPALAD.info: I0405 16:44:39.136433 10963 impala-beeswax-server.cc:172] query(): query=describe tab1 I0405 16:44:39.136504 10963 impala-beeswax-server.cc:515] query: Query { 01: query (string) = "describe tab1", 03: configuration (list) = list[0] { }, 04: hadoop_u

[Impala-CR](cdh5-trunk) IMPALA-3274: Always start Kudu for testing

2016-04-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3274: Always start Kudu for testing .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2690 To unsubscribe, visit http://gerrit.

Re: maven error while compiling the front end code

2016-04-05 Thread Pradeep Nayak
As for logs, I just see this : E0405 15:19:47.907562 10957 Analyzer.java:2210] Failed to load metadata for table: tab1 java.lang.NoClassDefFoundError: Could not initialize class com.cloudera.impala.util.HdfsCachingUtil CAUSED BY: ExecutionException: java.lang.NoClassDefFoundError: Could not initi

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-05 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/2543/10/be/src/util/decompress.cc File be/src/u

Re: maven error while compiling the front end code

2016-04-05 Thread Pradeep Nayak
Here is the strange part: Where we make changes to core-site.xml, re-deploy the stale client configs from the cloudera manager, and restart the affected services we get into the NoClassDefound state. In order to get past it we needed to restart impala dameons, catalog server etc again from the clo

[Impala-CR](cdh5-trunk) IMPALA-2198: Differentiate queries in exceptional states in web UI

2016-04-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-2198: Differentiate queries in exceptional states in web UI .. IMPALA-2198: Differentiate queries in exceptional states in web UI In order to make

[Impala-CR](cdh5-trunk) IMPALA-2198: Differentiate queries in exceptional states in web UI

2016-04-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-2198: Differentiate queries in exceptional states in web UI .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/2625/1/www/queries

[Toolchain-CR] Build Kudu from source

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new patch set (#3). Change subject: Build Kudu from source .. Build Kudu from source With this change any arbitrary Kudu hash can be built. Previously only a released version of Kudu could be used beca

[Toolchain-CR] Add TPC-H and TPC-DS

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new change for review. http://gerrit.cloudera.org:8080/2719 Change subject: Add TPC-H and TPC-DS .. Add TPC-H and TPC-DS TPC-H and DS aren't for build purposes. They are provided here as a convenien

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-05 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/2543/10/be/src/util/decompress.cc File be/src

[Toolchain-CR] Build Kudu from source

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has posted comments on this change. Change subject: Build Kudu from source .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/2679/1/functions.sh File functions.sh: Line 233: build_dist_package 2>&

[Toolchain-CR] Build Kudu from source

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new patch set (#2). Change subject: Build Kudu from source .. Build Kudu from source With this change any arbitrary Kudu hash can be built. Previously only a released version of Kudu could be used beca

[Impala-CR](cdh5-trunk) Disable Kudu on SLES and Debian

2016-04-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Disable Kudu on SLES and Debian .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2718 To unsubscribe, visit http://gerrit.clouder

[Impala-CR](cdh5-trunk) Disable Kudu on SLES and Debian

2016-04-05 Thread Casey Ching (Code Review)
Casey Ching has uploaded a new change for review. http://gerrit.cloudera.org:8080/2718 Change subject: Disable Kudu on SLES and Debian .. Disable Kudu on SLES and Debian Kudu won't work on these platforms yet. Change-Id: I38

[Impala-CR](cdh5-trunk) IMPALA-2645: Set mechUsing only on success

2016-04-05 Thread Henry Robinson (Code Review)
Hello Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2687 to look at the new patch set (#3). Change subject: IMPALA-2645: Set mechUsing only on success .. IMPALA-2645: S

[Impala-CR](cdh5-trunk) IMPALA-3283: Disable runtime filter tests for local filesystems

2016-04-05 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-3283: Disable runtime filter tests for local filesystems .. Patch Set 2: Code-Review+2 Rebase -- To view, visit http://gerrit.cloudera.org:8080/2688 To

[Impala-CR](cdh5-trunk) IMPALA-3283: Disable runtime filter tests for local filesystems

2016-04-05 Thread Henry Robinson (Code Review)
Henry Robinson has submitted this change and it was merged. Change subject: IMPALA-3283: Disable runtime filter tests for local filesystems .. IMPALA-3283: Disable runtime filter tests for local filesystems The runtime filter t

Re: maven error while compiling the front end code

2016-04-05 Thread Skye Wanderman-Milne
In case others run into similar issues, can you share how you resolved the NoClassDefFoundErrors? On Mon, Apr 4, 2016 at 10:21 PM, Henry Robinson wrote: > > > On 4 April 2016 at 17:55, Pradeep Nayak wrote: > >> Also just wanted to understand one more thing. >> http://www.cloudera.com/documentat

[Impala-CR](cdh5-trunk) IMPALA-3292. Kudu scanner should not fail if KeepAlive request fails

2016-04-05 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3292. Kudu scanner should not fail if KeepAlive request fails .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/2708/1//COMMIT_MSG Commit Mes

[Impala-CR](cdh5-trunk) Upgrade LLVM to 3.8.0

2016-04-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Upgrade LLVM to 3.8.0 .. Patch Set 7: Rebased, switched to using debug LLVM for debug build (dependent on http://gerrit.cloudera.org:8080/#/c/2658/) -- To vie

[Impala-CR](cdh5-trunk) Upgrade LLVM to 3.8.0

2016-04-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#7). Change subject: Upgrade LLVM to 3.8.0 .. Upgrade LLVM to 3.8.0 This requires various changes for Impala to be fully functional with the new version of LLVM. The original JIT was

Re: Fw: Debugging Impala code

2016-04-05 Thread Tim Armstrong
I agree with Jim's suggestion. On Tue, Apr 5, 2016 at 8:59 AM, Jim Apple wrote: > Sudarshan, you have asked, "Please help on list of test issues which > Nishidha has mentioned." > > In my personal opinion, it would help to tackle each issue separately, in > a separate thread or JIRA for each. Ho

Re: Fw: Debugging Impala code

2016-04-05 Thread Tim Armstrong
I don't think we've seen those test failures before. A lot of them look like calculations are returning different results, so those are probably related to the x86/PowerPC port. E.g. data-stream-test seems to be producing inconsistent hash values and expr-test seems to be producing incorrect result

Re: Fw: Debugging Impala code

2016-04-05 Thread Jim Apple
Sudarshan, you have asked, "Please help on list of test issues which Nishidha has mentioned." In my personal opinion, it would help to tackle each issue separately, in a separate thread or JIRA for each. How does that sound? On Mon, Apr 4, 2016 at 1:15 AM, Sudarshan Jagadale wrote: > Dear Jim,

Re: IOError while building Impala

2016-04-05 Thread hxu
Could you try not using user root? > On Apr 5, 2016, at 8:28 AM, Tim Armstrong wrote: > > Moving to apache incubator dev list. > > I don't see how that file could be missing unless a previous build step > failed. Are there earlier errors? Are you running the script with > permissions to write

Re: IOError while building Impala

2016-04-05 Thread Tim Armstrong
Moving to apache incubator dev list. I don't see how that file could be missing unless a previous build step failed. Are there earlier errors? Are you running the script with permissions to write the file? Is IMPALA_HOME set? On Tue, Apr 5, 2016 at 5:28 AM, valnci wrote: > Hi, > > When i try to