[Impala-ASF-CR] Move vim-specific config file to top level directory

2016-10-26 Thread Lars Volker (Code Review)
Lars Volker has abandoned this change. Change subject: Move vim-specific config file to top level directory .. Abandoned Let's remove the file altogether. -- To view, visit http://gerrit.cloudera.org:8080/4850 To

[Impala-ASF-CR] Remove vim plugin config file from .gitignore

2016-10-26 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4854 Change subject: Remove vim plugin config file from .gitignore .. Remove vim plugin config file from .gitignore Files in be/ get wiped out

[Impala-ASF-CR] Move vim-specific config file to top level directory

2016-10-25 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4850 Change subject: Move vim-specific config file to top level directory .. Move vim-specific config file to top level directory Files in be/

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-25 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2521: Add clustered hint to insert statements .. Patch Set 13: Alex, can you +2 this again? The tests got renamed and I didn't catch the syntax error when

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-25 Thread Lars Volker (Code Review)
Hello Marcel Kornacker, Internal Jenkins, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4745 to look at the new patch set (#13). Change subject: IMPALA-2521: Add clustered hint to insert statements

[Impala-ASF-CR] IMPALA-4086: Add benchmark for simple scheduler

2016-10-25 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#3). Change subject: IMPALA-4086: Add benchmark for simple scheduler .. IMPALA-4086: Add benchmark for simple scheduler Change-Id: I89ec1c6c1828bb0b86d1e13ce4dfc5a8df865c2e --- M

[Impala-ASF-CR] IMPALA-4086: Add benchmark for simple scheduler

2016-10-25 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: IMPALA-4086: Add benchmark for simple scheduler .. IMPALA-4086: Add benchmark for simple scheduler Change-Id: I89ec1c6c1828bb0b86d1e13ce4dfc5a8df865c2e --- M

[Impala-ASF-CR] IMPALA-4086: Add benchmark for simple scheduler

2016-10-25 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4086: Add benchmark for simple scheduler .. Patch Set 1: (3 comments) Thanks Tim for the review. I addressed your comments in PS2. I will rebase this

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-25 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2521: Add clustered hint to insert statements .. Patch Set 12: Rebase done in PS12 -- To view, visit http://gerrit.cloudera.org:8080/4745 To unsubscribe,

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-25 Thread Lars Volker (Code Review)
Hello Marcel Kornacker, Internal Jenkins, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4745 to look at the new patch set (#12). Change subject: IMPALA-2521: Add clustered hint to insert statements

[Impala-ASF-CR] IMPALA-4223: Handle truncated file read from HDFS cache

2016-10-25 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4223: Handle truncated file read from HDFS cache .. Patch Set 1: (1 comment) > (1 comment) > > Can we test this by having the test load metadata and

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-25 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2521: Add clustered hint to insert statements .. Patch Set 11: Fixed AnalyzeStmtsTest, also needs another rebase due to more Kudu related changes making

[Impala-ASF-CR] IMPALA-4223: Handle truncated file read from HDFS cache

2016-10-24 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4223: Handle truncated file read from HDFS cache .. Patch Set 1: > (1 comment) > > The change makes sense and we should get it in regardless, but >

[Impala-ASF-CR] IMPALA-4223: Handle truncated file read from HDFS cache

2016-10-24 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4828 Change subject: IMPALA-4223: Handle truncated file read from HDFS cache .. IMPALA-4223: Handle truncated file read from HDFS cache While

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-24 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2521: Add clustered hint to insert statements .. Patch Set 10: (1 comment) I added a test. I also had to rebase onto Dimitris' change, so now the diff

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-24 Thread Lars Volker (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4745 to look at the new patch set (#10). Change subject: IMPALA-2521: Add clustered hint to insert statements ..

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-24 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2521: Add clustered hint to insert statements .. Patch Set 9: (6 comments) Thanks for the comments, please see PS9.

[Impala-ASF-CR] Minor fixes to remove more "cloudera"s from the code.

2016-10-23 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Minor fixes to remove more "cloudera"s from the code. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-21 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#7). Change subject: IMPALA-2521: Add clustered hint to insert statements .. IMPALA-2521: Add clustered hint to insert statements This change introduces a clustered/noclustered hint for

[Impala-ASF-CR] Minor fixes to remove more "cloudera"s from the code.

2016-10-21 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Minor fixes to remove more "cloudera"s from the code. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4774/1//COMMIT_MSG Commit Message: Line 7:

[Impala-ASF-CR] IMPALA-4329: Prevent crash in scheduler when no backends are registered

2016-10-20 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#3). Change subject: IMPALA-4329: Prevent crash in scheduler when no backends are registered .. IMPALA-4329: Prevent crash in scheduler when no backends are registered The scheduler

[Impala-ASF-CR] IMPALA-4329: Prevent crash in scheduler when no backends are registered

2016-10-20 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: IMPALA-4329: Prevent crash in scheduler when no backends are registered .. IMPALA-4329: Prevent crash in scheduler when no backends are registered The scheduler

[Impala-ASF-CR] IMPALA-4329: Prevent crash in scheduler when no backends are registered

2016-10-20 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4776 Change subject: IMPALA-4329: Prevent crash in scheduler when no backends are registered .. IMPALA-4329: Prevent crash in scheduler when

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-19 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#5). Change subject: IMPALA-2521: Add clustered hint to insert statements .. IMPALA-2521: Add clustered hint to insert statements This change introduces a clustered/noclustered hint for

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-19 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2521: Add clustered hint to insert statements .. Patch Set 4: (8 comments) > (8 comments) > > Nice! Current PS looks good to me. Do you intent to add

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-19 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#4). Change subject: IMPALA-2521: Add clustered hint to insert statements .. IMPALA-2521: Add clustered hint to insert statements This change introduces a clustered/noclustered hint for

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-19 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2521: Add clustered hint to insert statements .. Patch Set 4: (24 comments) Thanks for the comments, please see PS4. As pointed out in one of the replies

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-19 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#3). Change subject: IMPALA-2521: Add clustered hint to insert statements .. IMPALA-2521: Add clustered hint to insert statements This change introduces a clustered/noclustered hint for

[Impala-ASF-CR] IMPALA-2521: Add clustered hint to insert statements

2016-10-18 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: IMPALA-2521: Add clustered hint to insert statements .. IMPALA-2521: Add clustered hint to insert statements This change introduces a clustered/noclustered hint for

[Impala-ASF-CR] PREVIEW IMPALA-2521: Add clustered hint to insert statements

2016-10-17 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: PREVIEW IMPALA-2521: Add clustered hint to insert statements .. Patch Set 1: In a first round I'd like to ask for feedback on the overall approach and the open

[Impala-ASF-CR] PREVIEW IMPALA-2521: Add clustered hint to insert statements

2016-10-17 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4745 Change subject: PREVIEW IMPALA-2521: Add clustered hint to insert statements .. PREVIEW IMPALA-2521: Add clustered hint to insert

[Impala-ASF-CR] IMPALA-2916: Add warning to query profile if debug build

2016-10-17 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2916: Add warning to query profile if debug build .. Patch Set 1: (2 comments) Thanks for the review, please see PS2.

[Impala-ASF-CR] IMPALA-2916: Add warning to query profile if debug build

2016-10-17 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: IMPALA-2916: Add warning to query profile if debug build .. IMPALA-2916: Add warning to query profile if debug build Change-Id:

[Impala-ASF-CR] IMPALA-3644 Make predicate order deterministic

2016-10-14 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-3644 Make predicate order deterministic .. Patch Set 4: (1 comment) Thanks for the review. Can you +2 again?

[Impala-ASF-CR] IMPALA-3644 Make predicate order deterministic

2016-10-14 Thread Lars Volker (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4671 to look at the new patch set (#5). Change subject: IMPALA-3644 Make predicate order deterministic .. IMPALA-3644 Make

[Impala-ASF-CR] IMPALA-3644 Make predicate order deterministic

2016-10-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-3644 Make predicate order deterministic .. Patch Set 4: This change now still passes on Java 7 and fixes all but 3 tests in PlannerTest. I will run the

[Impala-ASF-CR] IMPALA-3644 Make predicate order deterministic

2016-10-12 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#3). Change subject: IMPALA-3644 Make predicate order deterministic .. IMPALA-3644 Make predicate order deterministic This adds a tie-break to make sure that we sort predicates in a

[Impala-ASF-CR] IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo

2016-10-11 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo .. Patch Set 9: Exhaustive run finished successfully.

[Impala-ASF-CR] PREVIEW IMPALA-3644 Make predicate order deterministic

2016-10-10 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: PREVIEW IMPALA-3644 Make predicate order deterministic .. Patch Set 2: The tests have failed, there still seems to be more to fix. I added more comments to the

[Impala-ASF-CR] IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo

2016-10-10 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo .. Patch Set 8: (4 comments) http://gerrit.cloudera.org:8080/#/c/4187/8/be/src/util/redactor.detail.h

[Impala-ASF-CR] IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo

2016-10-10 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo .. Patch Set 8: Thanks everyone for following this. I would like to ask everyone for a final pass with

[Impala-ASF-CR] IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo

2016-10-10 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/4187/5/tests/comparison/cluster.py File

[Impala-ASF-CR] IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo

2016-10-10 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#8). Change subject: IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo .. IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo This change removes some of the occurrences of

[Impala-ASF-CR] IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo

2016-10-10 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#7). Change subject: IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo .. IMPALA-4047: Remove occurrences of 'CDH'/'cdh' from repo This change removes some of the occurrences of

[Impala-ASF-CR] PREVIEW IMPALA-3644 Make predicate order deterministic

2016-10-09 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: PREVIEW IMPALA-3644 Make predicate order deterministic .. PREVIEW IMPALA-3644 Make predicate order deterministic This adds a tie-break suggested by Alex in

[Impala-ASF-CR] PREVIEW IMPALA-3644 Make predicate order deterministic

2016-10-08 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: PREVIEW IMPALA-3644 Make predicate order deterministic .. Patch Set 1: > Thanks, Lars! Change looks good, let's wait for your validation. There were 4 failures,

[Impala-ASF-CR] PREVIEW IMPALA-3644 Make predicate order deterministic

2016-10-07 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4671 Change subject: PREVIEW IMPALA-3644 Make predicate order deterministic .. PREVIEW IMPALA-3644 Make predicate order deterministic This

[Impala-ASF-CR] IMPALA-4047: DO NOT SUBMIT Highlight all occurrences of CDH/cdh

2016-10-06 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4047: DO NOT SUBMIT Highlight all occurrences of CDH/cdh .. Patch Set 5: > Any updates on this? I'm currently focusing on IMPALA-4223, after that I will

[Impala-ASF-CR] IMPALA-4234: Remove astyle config file, looks outdated.

2016-09-30 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4578 Change subject: IMPALA-4234: Remove astyle config file, looks outdated. .. IMPALA-4234: Remove astyle config file, looks outdated.

[Impala-ASF-CR] Add vim-specific files to .gitignore

2016-09-30 Thread Lars Volker (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4562 to look at the new patch set (#2). Change subject: Add vim-specific files to .gitignore .. Add vim-specific files

[Impala-ASF-CR] IMPALA-3342, IMPALA-3920: Adding thread counters to obtain thread stats per scanner thread, and to measure time spent in per-fragment-executor Open() and ProcessBuildInputAsync calls

2016-09-29 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-3342, IMPALA-3920: Adding thread counters to obtain thread stats per scanner thread, and to measure time spent in per-fragment-executor Open() and ProcessBuildInputAsync calls

[Impala-ASF-CR] IMPALA-4047: DO NOT SUBMIT Highlight all occurrences of CDH/cdh

2016-09-29 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4047: DO NOT SUBMIT Highlight all occurrences of CDH/cdh .. Patch Set 5: (24 comments) Thanks for all the help so far. I created a bunch of upstream Jiras

[Impala-ASF-CR] IMPALA-4086: Add benchmark for simple scheduler

2016-09-28 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4554 Change subject: IMPALA-4086: Add benchmark for simple scheduler .. IMPALA-4086: Add benchmark for simple scheduler Change-Id:

[Impala-ASF-CR] IMPALA-4057 and IMPALA-4050 Support starting webserver specified by hostname or "127.0.0.1"

2016-09-28 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4057 and IMPALA-4050 Support starting webserver specified by hostname or "127.0.0.1" .. Patch Set 1: There now seem to be 5 different Gerrit changes

[Impala-ASF-CR] IMPALA-4057 and IMPALA-4050 Support starting webserver specified by hostname or "127.0.0.1"

2016-09-28 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4057 and IMPALA-4050 Support starting webserver specified by hostname or "127.0.0.1" .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-4057 and IMPALA-4050 Support starting webserver specified by hostname or "127.0.0.1"

2016-09-28 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4057 and IMPALA-4050 Support starting webserver specified by hostname or "127.0.0.1" .. Patch Set 1: (6 comments)

[Impala-ASF-CR] IMPALA-4047: DO NOT SUBMIT Highlight all occurrences of CDH/cdh

2016-09-26 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4047: DO NOT SUBMIT Highlight all occurrences of CDH/cdh .. Patch Set 3: (1 comment) > Any updates? Not yet, will look at this again this week, hopefully

[Impala-ASF-CR] IMPALA-4199: Add 'SNAPSHOT' to Impala version

2016-09-26 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4539 Change subject: IMPALA-4199: Add 'SNAPSHOT' to Impala version .. IMPALA-4199: Add 'SNAPSHOT' to Impala version Unreleased versions should

[Impala-ASF-CR] IMPALA-4193: Warn when benchmarks run with sub-optimal CPU settings

2016-09-23 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4193: Warn when benchmarks run with sub-optimal CPU settings .. Patch Set 7: (1 comment)

[Impala-ASF-CR] IMPALA-4193: Warn when benchmarks run with sub-optimal CPU settings

2016-09-23 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4193: Warn when benchmarks run with sub-optimal CPU settings .. Patch Set 5: (3 comments) Thanks for the review, please see PS5.

[Impala-ASF-CR] IMPALA-4193: Warn when benchmarks run with sub-optimal CPU settings

2016-09-23 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4193: Warn when benchmarks run with sub-optimal CPU settings .. Patch Set 3: (6 comments) Thanks for the review. Please see PS3.

[Impala-ASF-CR] IMPALA-4193: Warn when benchmarks run with sub-optimal CPU settings

2016-09-23 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#3). Change subject: IMPALA-4193: Warn when benchmarks run with sub-optimal CPU settings .. IMPALA-4193: Warn when benchmarks run with sub-optimal CPU settings Change-Id:

[Impala-ASF-CR] WIP: Issue warnings when benchmarks run with sub-optimal system settings

2016-09-23 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: WIP: Issue warnings when benchmarks run with sub-optimal system settings .. WIP: Issue warnings when benchmarks run with sub-optimal system settings This is

[Impala-ASF-CR] WIP: Issue warnings when benchmarks run with sub-optimal system settings

2016-09-23 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4528 Change subject: WIP: Issue warnings when benchmarks run with sub-optimal system settings .. WIP: Issue warnings when benchmarks run with

[Impala-ASF-CR] IMPALA-4117: Factor simple scheduler test code into own files

2016-09-20 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4486 Change subject: IMPALA-4117: Factor simple scheduler test code into own files .. IMPALA-4117: Factor simple scheduler test code into own

[Impala-ASF-CR] IMPALA-4116: Remove 'cdh' from version string again

2016-09-16 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4116: Remove 'cdh' from version string again .. Patch Set 1: As a non-committer I cannot submit this. Can one of you please submit and push it to the asf

[Impala-ASF-CR] Make gen build version.py resilient to a failing git rev-parse

2016-09-16 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Make gen_build_version.py resilient to a failing git rev-parse .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/4411/8/bin/save-version.sh File

[Impala-ASF-CR] IMPALA-4116: Remove 'cdh' from version string again

2016-09-16 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4439 Change subject: IMPALA-4116: Remove 'cdh' from version string again .. IMPALA-4116: Remove 'cdh' from version string again The rebase of

[Impala-ASF-CR] IMPALA-4116: Remove 'cdh' from version string

2016-09-14 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: IMPALA-4116: Remove 'cdh' from version string .. IMPALA-4116: Remove 'cdh' from version string Change-Id: I7754538a23e73dcdebc6e3df509f357cbe03198c --- M

[Impala-ASF-CR] Make gen build version.py resilient to a failing git rev-parse

2016-09-14 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Make gen_build_version.py resilient to a failing git rev-parse .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4411/3/bin/gen_build_version.py

[Impala-ASF-CR] Make gen build version.py resilient to a failing git rev-parse

2016-09-14 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Make gen_build_version.py resilient to a failing git rev-parse .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/4411/2/bin/gen_build_version.py

[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-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-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-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,

<    4   5   6   7   8   9