[Impala-ASF-CR] IMPALA-3973: add position and occurrence to instr()

2016-09-12 Thread Zoltan Ivanfi (Code Review)
Hello Lars Volker, Matthew Jacobs, Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4094 to look at the new patch set (#15). Change subject: IMPALA-3973: add position and occurrence to instr()

[Impala-ASF-CR] IMPALA-1616: Improve the Memory Limit Exceeded error report

2016-09-12 Thread Attila Jeges (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4335 to look at the new patch set (#4). Change subject: IMPALA-1616: Improve the Memory Limit Exceeded error report

[Impala-ASF-CR] IMPALA-1616: Improve the Memory Limit Exceeded error report

2016-09-12 Thread Attila Jeges (Code Review)
Attila Jeges has uploaded a new patch set (#4). Change subject: IMPALA-1616: Improve the Memory Limit Exceeded error report .. IMPALA-1616: Improve the Memory Limit Exceeded error report The error report has been changed to

[Impala-ASF-CR] IMPALA-4111: backend death tests should not produce minidumps

2016-09-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4111: backend death tests should not produce minidumps .. Patch Set 4: Code-Review+1 Carry +1 -- To view, visit http://gerrit.cloudera.org:8080/4353 To

[Impala-ASF-CR] IMPALA-4111: backend death tests should not produce minidumps

2016-09-12 Thread Tim Armstrong (Code Review)
Hello Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4353 to look at the new patch set (#4). Change subject: IMPALA-4111: backend death tests should not produce minidumps ..

[Impala-ASF-CR] IMPALA-1616: Improve the Memory Limit Exceeded error report

2016-09-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-1616: Improve the Memory Limit Exceeded error report .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4335 To unsubscribe,

[Impala-ASF-CR] IMPALA-4028: Trim sentry config file path spaces while impala start.

2016-09-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4028: Trim sentry config file path spaces while impala start. .. Patch Set 3: How did you come across this? Did you start Impala from the command line

[Impala-ASF-CR] IMPALA-4091: Fix backend unit to log in logs/be tests.

2016-09-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4091: Fix backend unit to log in logs/be_tests. .. Patch Set 1: Thanks for fixing this. While we are at it, why don't we create a common library for the

[Impala-ASF-CR] IMPALA-4008: don't bake in hash table and hash join pointers

2016-09-12 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4008: don't bake in hash table and hash join pointers .. Patch Set 3: (8 comments) http://gerrit.cloudera.org:8080/#/c/4326/3/be/src/exec/hash-table.h File

[Impala-ASF-CR] IMPALA-4026: Implement double-buffering for BlockingQueue.

2016-09-12 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4026: Implement double-buffering for BlockingQueue. .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/4350/2/be/src/util/blocking-queue.h

[Impala-ASF-CR] IMPALA-3918: Fix straggler Cloudera -> ASF license headers

2016-09-12 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3918: Fix straggler Cloudera -> ASF license headers .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4386 To unsubscribe,

[Impala-ASF-CR] IMPALA-4014: PARALLEL HEADERS ONLY: Introduce query-wide execution state.

2016-09-12 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded a new patch set (#3). Change subject: IMPALA-4014: PARALLEL HEADERS ONLY: Introduce query-wide execution state. .. IMPALA-4014: PARALLEL HEADERS ONLY: Introduce query-wide execution state. This patch

[Impala-ASF-CR] IMPALA-3918: Fix straggler Cloudera -> ASF license headers

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: IMPALA-3918: Fix straggler Cloudera -> ASF license headers .. Patch Set 1: lgtm, though it might be a good idea to document the reasoning that the "relicense" is

[Impala-ASF-CR] IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs.

2016-09-12 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#2). Change subject: IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs. .. IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs. While I'm here, fix as many of the warnings as

[Impala-ASF-CR] IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs.

2016-09-12 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110: PREVIEW: Make RAT run on Impala tarballs. .. Patch Set 1: (3 comments) > (5 comments) > > looks pretty reasonable. Might want to do a surface level

[Impala-ASF-CR] Fix typo in buildall.sh introduced in IMPALA-4006

2016-09-12 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: Fix typo in buildall.sh introduced in IMPALA-4006 .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4382 To unsubscribe, visit

[Impala-ASF-CR] Fix typo in buildall.sh introduced in IMPALA-4006

2016-09-12 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: Fix typo in buildall.sh introduced in IMPALA-4006 .. Fix typo in buildall.sh introduced in IMPALA-4006 The typo resulted in a silent failure: an

[Impala-ASF-CR] IMPALA-4099: Fix the error message while loading UDFs with no JARs

2016-09-12 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4099: Fix the error message while loading UDFs with no JARs .. Patch Set 1: (5 comments) Thanks Alex for the review.

[Impala-ASF-CR] IMPALA-4014: PARALLEL HEADERS ONLY: Introduce query-wide execution state.

2016-09-12 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4014: PARALLEL HEADERS ONLY: Introduce query-wide execution state. .. Patch Set 4: (27 comments) Thanks for the review Lars. I've made the changes.

[Impala-ASF-CR] IMPALA-4111: backend death tests should not produce minidumps

2016-09-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4111: backend death tests should not produce minidumps .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-4111: backend death tests should not produce minidumps

2016-09-12 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4111: backend death tests should not produce minidumps .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-4099: Fix the error message while loading UDFs with no JARs

2016-09-12 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#2). Change subject: IMPALA-4099: Fix the error message while loading UDFs with no JARs .. IMPALA-4099: Fix the error message while loading UDFs with no JARs Currently we

[Impala-ASF-CR] IMPALA-4099: Fix the error message while loading UDFs with no JARs

2016-09-12 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4099: Fix the error message while loading UDFs with no JARs .. Patch Set 2: Code-Review+1 Henry, Marcel, can one of you sign off? -- To view, visit

[Impala-ASF-CR] IMPALA-4091: Fix backend unit to log in logs/be tests.

2016-09-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4091: Fix backend unit to log in logs/be_tests. .. Patch Set 2: (9 comments) Made a first pass. I'm still building the change to validate it myself.

[Impala-ASF-CR] IMPALA-4096: Allow clean.sh to work from snapshots

2016-09-12 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: IMPALA-4096: Allow clean.sh to work from snapshots .. IMPALA-4096: Allow clean.sh to work from snapshots buildall.sh calls bin/clean.sh, which fails when

[Impala-ASF-CR] IMPALA-3823: Add timer to measure Parquet footer reads

2016-09-12 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3823: Add timer to measure Parquet footer reads .. Patch Set 3: (5 comments)

[Impala-ASF-CR] IMPALA-4028: Trim sentry config file path spaces while impala start.

2016-09-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4028: Trim sentry config file path spaces while impala start. .. Patch Set 3: I don't think we should go ahead with the trimming solution for the

[Impala-ASF-CR] IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

2016-09-12 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. Patch Set 4: (27 comments)

[Impala-ASF-CR] IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

2016-09-12 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has uploaded a new patch set (#5). Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. IMPALA-3902: Scheduler improvements for running multiple

[Impala-ASF-CR] IMPALA-4099: Fix the error message while loading UDFs with no JARs

2016-09-12 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4099: Fix the error message while loading UDFs with no JARs .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4365 To

[Impala-ASF-CR] IMPALA-3918: Fix straggler Cloudera -> ASF license headers

2016-09-12 Thread Jim Apple (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4386 to look at the new patch set (#2). Change subject: IMPALA-3918: Fix straggler Cloudera -> ASF license headers ..

[Impala-ASF-CR] IMPALA-3918: Fix straggler Cloudera -> ASF license headers

2016-09-12 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3918: Fix straggler Cloudera -> ASF license headers .. Patch Set 2: Code-Review+2 > lgtm, though it might be a good idea to document the reasoning that >

[Impala-ASF-CR] IMPALA-4091: Fix backend unit to log in logs/be tests.

2016-09-12 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4091: Fix backend unit to log in logs/be_tests. .. Patch Set 1: I decided to add a macro for the common main() instead of going the library route. The

[Impala-ASF-CR] IMPALA-3823: Add timer to measure Parquet footer reads

2016-09-12 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3823: Add timer to measure Parquet footer reads .. Patch Set 3: (18 comments) http://gerrit.cloudera.org:8080/#/c/4371/2//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-4026: Implement double-buffering for BlockingQueue.

2016-09-12 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4026: Implement double-buffering for BlockingQueue. .. Patch Set 3: (6 comments) http://gerrit.cloudera.org:8080/#/c/4350/3/be/src/util/blocking-queue.h

[Impala-ASF-CR] IMPALA-4026: Implement double-buffering for BlockingQueue.

2016-09-12 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new patch set (#3). Change subject: IMPALA-4026: Implement double-buffering for BlockingQueue. .. IMPALA-4026: Implement double-buffering for BlockingQueue. With recent changes to improve the parquet