[Impala-ASF-CR] Impala-992: Rerun past queries from history in shell

2017-08-15 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Impala-992: Rerun past queries from history in shell .. Patch Set 2: (9 comments) http://gerrit.cloudera.org:8080/#/c/7674/2//COMMIT_MSG Commit Message: PS2,

[Impala-ASF-CR] IMPALA-5109: Increase range of backend latency histogram

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5109: Increase range of backend latency histogram .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1059/ -- To

[Impala-ASF-CR] Fix link to Hadoop ADLS page

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Fix link to Hadoop ADLS page .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-docs-submit/147/ -- To view, visit

[Impala-ASF-CR] [DOCS] Fold some lines

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Fold some lines .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-docs-submit/148/ -- To view, visit

[Impala-ASF-CR] IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2 .. Patch Set 2: (2 comments) Looks good, just had minor comments.

[Impala-ASF-CR] IMPALA-5644: Reject queries if min reservation is too large

2017-08-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/7678 Change subject: IMPALA-5644: Reject queries if min reservation is too large .. IMPALA-5644: Reject queries if min reservation is too

[Impala-ASF-CR] IMPALA-5787: Dropped status in KuduTableSink::Send()

2017-08-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5787: Dropped status in KuduTableSink::Send() .. Patch Set 1: Code-Review+2 (1 comment) I'm OK with making this change now, but there are a bunch of

[Impala-ASF-CR] IMPALA-5477: Fix minidump-2-core tool

2017-08-15 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/7677 Change subject: IMPALA-5477: Fix minidump-2-core tool .. IMPALA-5477: Fix minidump-2-core tool This change bumps the toolchain version to

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-15 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7581/2/bin/impala-config.sh File

[Impala-ASF-CR] Impala-992: Rerun past queries from history in shell

2017-08-15 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: Impala-992: Rerun past queries from history in shell .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7674/2/shell/impala_shell.py File

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-15 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7581/2/bin/impala-config.sh File

[Impala-ASF-CR] IMPALA-5477: Fix minidump-2-core tool

2017-08-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5477: Fix minidump-2-core tool .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7677 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5776: Write partial tuple to the correct mempool

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5776: Write partial tuple to the correct mempool .. Patch Set 2: (6 comments) This is already easier to understand but I'm hoping we can avoid adding

[Impala-ASF-CR] Fix link to Hadoop ADLS page

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: Fix link to Hadoop ADLS page .. Fix link to Hadoop ADLS page At the time I added the original link, the URL with /current2/ was correct. Now

[Impala-ASF-CR] Fix link to Hadoop ADLS page

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Fix link to Hadoop ADLS page .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7672 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5109: Increase range of backend latency histogram

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5109: Increase range of backend latency histogram .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7676 To unsubscribe,

[Impala-ASF-CR] IMPALA-4833: Compute precise per-host reservation size

2017-08-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4833: Compute precise per-host reservation size .. Patch Set 8: (6 comments) http://gerrit.cloudera.org:8080/#/c/7630/8/be/src/scheduling/query-schedule.h

[Impala-ASF-CR] Impala-992: Rerun past queries from history in shell

2017-08-15 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: Impala-992: Rerun past queries from history in shell .. Patch Set 2: (3 comments) Could you add some tests for this too? Have a look at

[Impala-ASF-CR] IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-4794: Grouping distinct agg plan robust to data skew

2017-08-15 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4794: Grouping distinct agg plan robust to data skew .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7643 To unsubscribe,

[Impala-ASF-CR] IMPALA-5776: Write partial tuple to the correct mempool

2017-08-15 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5776: Write partial tuple to the correct mempool .. Patch Set 4: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-5598: Fix excessive dumping in MemLimitExceeded

2017-08-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5598: Fix excessive dumping in MemLimitExceeded .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/7597/3/be/src/runtime/mem-tracker.cc File

[Impala-ASF-CR] IMPALA-5769: Add periodic minidump cleanup

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5769: Add periodic minidump cleanup .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/7605/2/tests/custom_cluster/test_breakpad.py File

[Impala-ASF-CR] IMPALA-5116: Remove deprecated hash * types in gutil

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5116: Remove deprecated hash_* types in gutil .. IMPALA-5116: Remove deprecated hash_* types in gutil The following class templates are

[Impala-ASF-CR] IMPALA-5116: Remove deprecated hash * types in gutil

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5116: Remove deprecated hash_* types in gutil .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7414 To unsubscribe,

[Impala-ASF-CR] Impala-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded a new change for review. http://gerrit.cloudera.org:8080/7674 Change subject: Impala-992: Rerun past queries from history in shell .. Impala-992: Rerun past queries from history in shell This patch

[Impala-ASF-CR] [DOCS] Fold some lines

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: [DOCS] Fold some lines .. [DOCS] Fold some lines Putting on a separate line causes an extra leading blank line in PDF output. (I think

[Impala-ASF-CR] [DOCS] Fold some lines

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Fold some lines .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7417 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3931: arbitrary fixed-size uda intermediate types

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3931: arbitrary fixed-size uda intermediate types .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/7526/8/be/src/codegen/llvm-codegen.cc

[Impala-ASF-CR] IMPALA-5776: Write partial tuple to the correct mempool

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5776: Write partial tuple to the correct mempool .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7639/4/be/src/exec/hdfs-text-scanner.h

[Impala-ASF-CR] IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled

2017-08-15 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled .. Patch Set 3: Code-Review+2 Rebased, carrying Tim's +2. -- To view, visit

[Impala-ASF-CR] IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-5776: Write partial tuple to the correct mempool

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5776: Write partial tuple to the correct mempool .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7639/4/be/src/exec/hdfs-text-scanner.h

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded a new patch set (#3). Change subject: IMPALA-992: Rerun past queries from history in shell .. IMPALA-992: Rerun past queries from history in shell This patch adds a new command "rerun" and shortcut "@"

[Impala-ASF-CR] IMPALA-5309: [DOCS] Add TABLESAMPLE clause to SELECT statement

2017-08-15 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: IMPALA-5309: [DOCS] Add TABLESAMPLE clause to SELECT statement .. Patch Set 1: For my TABLESAMPLE examples, I used a trivial amount of data just to make it

[Impala-ASF-CR] IMPALA-5598: Fix excessive dumping in MemLimitExceeded

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5598: Fix excessive dumping in MemLimitExceeded .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1063/ -- To

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-992: Rerun past queries from history in shell .. Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/7674/4/shell/impala_shell.py File

[Impala-ASF-CR] IMPALA-5787: Dropped status in KuduTableSink::Send()

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5787: Dropped status in KuduTableSink::Send() .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1064/ -- To

[Impala-ASF-CR] IMPALA-5316: [DOCS] Document last day() function

2017-08-15 Thread John Russell (Code Review)
John Russell has uploaded a new change for review. http://gerrit.cloudera.org:8080/7682 Change subject: IMPALA-5316: [DOCS] Document last_day() function .. IMPALA-5316: [DOCS] Document last_day() function Change-Id:

[Impala-ASF-CR] IMPALA-5776: Write partial tuple to the correct mempool

2017-08-15 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-5776: Write partial tuple to the correct mempool .. Patch Set 4: (2 comments)

[Impala-ASF-CR] IMPALA-5109: Increase range of backend latency histogram

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5109: Increase range of backend latency histogram .. IMPALA-5109: Increase range of backend latency histogram Tiny change to increase

[Impala-ASF-CR] IMPALA-5109: Increase range of backend latency histogram

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5109: Increase range of backend latency histogram .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7676 To

[Impala-ASF-CR] IMPALA-5598: Fix excessive dumping in MemLimitExceeded

2017-08-15 Thread Joe McDonnell (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7597 to look at the new patch set (#4). Change subject: IMPALA-5598: Fix excessive dumping in MemLimitExceeded ..

[Impala-ASF-CR] IMPALA-5598: Fix excessive dumping in MemLimitExceeded

2017-08-15 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. Change subject: IMPALA-5598: Fix excessive dumping in MemLimitExceeded .. Patch Set 3: (2 comments) Also rebased to the latest.

[Impala-ASF-CR] IMPALA-3931: arbitrary fixed-size uda intermediate types

2017-08-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3931: arbitrary fixed-size uda intermediate types .. Patch Set 12: (9 comments) http://gerrit.cloudera.org:8080/#/c/7526/12/be/src/codegen/codegen-anyval.cc

[Impala-ASF-CR] IMPALA-5769: Add periodic minidump cleanup

2017-08-15 Thread Lars Volker (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7605 to look at the new patch set (#3). Change subject: IMPALA-5769: Add periodic minidump cleanup .. IMPALA-5769: Add

[Impala-ASF-CR] IMPALA-3200: More end-to-end tests

2017-08-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3200: More end-to-end tests .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7668/2/tests/verifiers/metric_verifier.py File

[Impala-ASF-CR] IMPALA-5769: Add periodic minidump cleanup

2017-08-15 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-5769: Add periodic minidump cleanup .. Patch Set 2: (4 comments) Thanks for the reviews. Please see PS3.

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded a new patch set (#5). Change subject: IMPALA-992: Rerun past queries from history in shell .. IMPALA-992: Rerun past queries from history in shell This patch adds a new command "rerun" and shortcut "@"

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded a new patch set (#5). Change subject: IMPALA-992: Rerun past queries from history in shell .. IMPALA-992: Rerun past queries from history in shell This patch adds a new command "rerun" and shortcut "@"

[Impala-ASF-CR] IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2

2017-08-15 Thread Henry Robinson (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7675 to look at the new patch set (#3). Change subject: IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2 ..

[Impala-ASF-CR] IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2

2017-08-15 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2 .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-5800: Configure Squeasel's cipher suite and TLS version

2017-08-15 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/7679 Change subject: IMPALA-5800: Configure Squeasel's cipher suite and TLS version .. IMPALA-5800: Configure Squeasel's cipher suite and

[Impala-ASF-CR] IMPALA-5598: Fix excessive dumping in MemLimitExceeded

2017-08-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5598: Fix excessive dumping in MemLimitExceeded .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7597 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. Change subject: IMPALA-992: Rerun past queries from history in shell .. Patch Set 4: (12 comments) http://gerrit.cloudera.org:8080/#/c/7674/2//COMMIT_MSG Commit Message: PS2,

[Impala-ASF-CR] IMPALA-4833: Compute precise per-host reservation size, pt2

2017-08-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/7681 Change subject: IMPALA-4833: Compute precise per-host reservation size, pt2 .. IMPALA-4833: Compute precise per-host reservation size,

[Impala-ASF-CR] IMPALA-4833: Compute precise per-host reservation size, pt2

2017-08-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4833: Compute precise per-host reservation size, pt2 .. Patch Set 1: Code-Review+1 This looks good to me, but let's still discuss. -- To view, visit

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-992: Rerun past queries from history in shell .. Patch Set 7: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-5477: Fix minidump-2-core tool

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5477: Fix minidump-2-core tool .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1061/ -- To view, visit

[Impala-ASF-CR] IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. Change subject: IMPALA-992: Rerun past queries from history in shell .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/7674/4/shell/impala_shell.py File

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-992: Rerun past queries from history in shell .. Patch Set 5: (5 comments) http://gerrit.cloudera.org:8080/#/c/7674/5//COMMIT_MSG Commit Message: Line 9:

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded a new patch set (#7). Change subject: IMPALA-992: Rerun past queries from history in shell .. IMPALA-992: Rerun past queries from history in shell This patch adds a new command "rerun" and a shortcut

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. Change subject: IMPALA-992: Rerun past queries from history in shell .. Patch Set 5: (5 comments) http://gerrit.cloudera.org:8080/#/c/7674/5//COMMIT_MSG Commit Message: Line 9:

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. Change subject: IMPALA-992: Rerun past queries from history in shell .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/7674/7/tests/shell/test_shell_interactive.py

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7674 to look at the new patch set (#8). Change subject: IMPALA-992: Rerun past queries from history in shell ..

[Impala-ASF-CR] IMPALA-5644: Reject queries if min reservation is too large

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5644: Reject queries if min reservation is too large .. Patch Set 1: (4 comments) The overall approach makes sense to me. Commented on the aspects of it

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded a new patch set (#4). Change subject: IMPALA-992: Rerun past queries from history in shell .. IMPALA-992: Rerun past queries from history in shell This patch adds a new command "rerun" and shortcut "@"

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded a new patch set (#4). Change subject: IMPALA-992: Rerun past queries from history in shell .. IMPALA-992: Rerun past queries from history in shell This patch adds a new command "rerun" and shortcut "@"

[Impala-ASF-CR] IMPALA-5309: [DOCS] Add TABLESAMPLE clause to SELECT statement

2017-08-15 Thread John Russell (Code Review)
John Russell has uploaded a new change for review. http://gerrit.cloudera.org:8080/7680 Change subject: IMPALA-5309: [DOCS] Add TABLESAMPLE clause to SELECT statement .. IMPALA-5309: [DOCS] Add TABLESAMPLE clause to SELECT

[Impala-ASF-CR] IMPALA-5769: Add periodic minidump cleanup

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5769: Add periodic minidump cleanup .. Patch Set 3: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2 .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7675 To unsubscribe,

[Impala-ASF-CR] IMPALA-3200: More end-to-end tests

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#3). Change subject: IMPALA-3200: More end-to-end tests .. IMPALA-3200: More end-to-end tests * Test that query startup fails if there is insufficient reservation on one backend. *

[Impala-ASF-CR] IMPALA-3200: More end-to-end tests

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3200: More end-to-end tests .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7668/2/tests/verifiers/metric_verifier.py File

[Impala-ASF-CR] IMPALA-2615: support [[nodiscard]] on Status

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-2615: support [[nodiscard]] on Status .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/7253/9/be/src/util/runtime-profile.cc File

[Impala-ASF-CR] IMPALA-2615: support [[nodiscard]] on Status

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-2615: support [[nodiscard]] on Status .. Patch Set 9: Yeah we could drop it for Status return values. We would keep it for bool or other return values

[Impala-ASF-CR] IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled .. Patch Set 3: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-992: Rerun past queries from history in shell .. Patch Set 9: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7674 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5769: Add periodic minidump cleanup

2017-08-15 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-5769: Add periodic minidump cleanup .. Patch Set 4: Code-Review+2 Needed a rebase. Carrying Tim's +2. -- To view, visit

[Impala-ASF-CR] IMPALA-5769: Add periodic minidump cleanup

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5769: Add periodic minidump cleanup .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1070/ -- To view, visit

[Impala-ASF-CR] IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2 .. Patch Set 3: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-3200: More end-to-end tests

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3200: More end-to-end tests .. Patch Set 5: Code-Review+2 rebase -- To view, visit http://gerrit.cloudera.org:8080/7668 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5800: Configure Squeasel's cipher suite and TLS version

2017-08-15 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5800: Configure Squeasel's cipher suite and TLS version .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/7679/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-5800: Configure Squeasel's cipher suite and TLS version

2017-08-15 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new patch set (#2). Change subject: IMPALA-5800: Configure Squeasel's cipher suite and TLS version .. IMPALA-5800: Configure Squeasel's cipher suite and TLS version * Import Squeasel from

[Impala-ASF-CR] IMPALA-5800: Configure Squeasel's cipher suite and TLS version

2017-08-15 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5800: Configure Squeasel's cipher suite and TLS version .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-3200: More end-to-end tests

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3200: More end-to-end tests .. Patch Set 5: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1069/ -- To view, visit

[Impala-ASF-CR] IMPALA-2615: support [[nodiscard]] on Status

2017-08-15 Thread Tim Armstrong (Code Review)
Hello Jim Apple, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7253 to look at the new patch set (#11). Change subject: IMPALA-2615: support [[nodiscard]] on Status ..

[Impala-ASF-CR] IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5775: Allow shell to support TLSv1, v1.1 and v1.2 .. Patch Set 3: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1067/ -- To

[Impala-ASF-CR] IMPALA-2615: support [[nodiscard]] on Status

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-2615: support [[nodiscard]] on Status .. Patch Set 11: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1068/ -- To view,

[Impala-ASF-CR] IMPALA-2615: support [[nodiscard]] on Status

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-2615: support [[nodiscard]] on Status .. Patch Set 11: Code-Review+2 rebase -- To view, visit http://gerrit.cloudera.org:8080/7253 To unsubscribe,

[Impala-ASF-CR] IMPALA-5316: [DOCS] Document last day() function

2017-08-15 Thread Greg Rahn (Code Review)
Greg Rahn has posted comments on this change. Change subject: IMPALA-5316: [DOCS] Document last_day() function .. Patch Set 1: Code-Review+1 LGTM. Thanks! -- To view, visit http://gerrit.cloudera.org:8080/7682 To

[Impala-ASF-CR] IMPALA-3931: arbitrary fixed-size uda intermediate types

2017-08-15 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7526 to look at the new patch set (#13). Change subject: IMPALA-3931: arbitrary fixed-size uda intermediate types

[Impala-ASF-CR] IMPALA-3200: More end-to-end tests

2017-08-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#4). Change subject: IMPALA-3200: More end-to-end tests .. IMPALA-3200: More end-to-end tests * Test that query startup fails if there is insufficient reservation on one backend. *

[Impala-ASF-CR] IMPALA-5477: Fix minidump-2-core tool

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5477: Fix minidump-2-core tool .. IMPALA-5477: Fix minidump-2-core tool This change bumps the toolchain version to include a fix for

[Impala-ASF-CR] IMPALA-5477: Fix minidump-2-core tool

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5477: Fix minidump-2-core tool .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7677 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7674 to look at the new patch set (#9). Change subject: IMPALA-992: Rerun past queries from history in shell ..

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded a new patch set (#9). Change subject: IMPALA-992: Rerun past queries from history in shell .. IMPALA-992: Rerun past queries from history in shell This patch adds a new command "rerun" and a shortcut

[Impala-ASF-CR] IMPALA-992: Rerun past queries from history in shell

2017-08-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. Change subject: IMPALA-992: Rerun past queries from history in shell .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/7674/8/tests/shell/test_shell_interactive.py

[Impala-ASF-CR] IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled

2017-08-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4737: Prevent SIGUSR1 from killing daemons when minidumps are disabled .. IMPALA-4737: Prevent SIGUSR1 from killing daemons when

[Impala-ASF-CR] IMPALA-5316: [DOCS] Document last day() function

2017-08-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5316: [DOCS] Document last_day() function .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7682 To unsubscribe, visit

  1   2   >