[Impala-ASF-CR] IMPALA-5586: Null-aware anti-join can take a long time to cancel

2017-07-24 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel .. IMPALA-5586: Null-aware anti-join can take a long time to cancel Queries with a null

[Impala-ASF-CR] IMPALA-5586: Null-aware anti-join can take a long time to cancel

2017-07-24 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7393 To unsubsc

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. IMPALA-5615: Fix compute incremental stats for general partition exprs

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 9: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/737

[Impala-ASF-CR] IMPALA-5709: Remove mini-impala-cluster

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5709: Remove mini-impala-cluster .. IMPALA-5709: Remove mini-impala-cluster Remove untested / unused mini-impala-cluster binary. Change-

[Impala-ASF-CR] IMPALA-5709: Remove mini-impala-cluster

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5709: Remove mini-impala-cluster .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7488 To unsubscribe, visit http://g

[Impala-ASF-CR] Populate OffsetIndex and ColumnIndex of a row group and Filter pages

2017-07-24 Thread Pooja Nilangekar (Code Review)
Pooja Nilangekar has posted comments on this change. Change subject: Populate OffsetIndex and ColumnIndex of a row_group and Filter pages .. Patch Set 2: (20 comments) http://gerrit.cloudera.org:8080/#/c/7465/1/be/src/exec/hd

[Impala-ASF-CR] Populate OffsetIndex and ColumnIndex of a row group and Filter pages

2017-07-24 Thread Pooja Nilangekar (Code Review)
Pooja Nilangekar has uploaded a new change for review. http://gerrit.cloudera.org:8080/7465 Change subject: Populate OffsetIndex and ColumnIndex of a row_group and Filter pages .. Populate OffsetIndex and ColumnIndex of a row

[Impala-ASF-CR] IMPALA-5716: Don't delete cmake modules/* when enabling distcc

2017-07-24 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/7493 Change subject: IMPALA-5716: Don't delete cmake_modules/* when enabling distcc .. IMPALA-5716: Don't delete cmake_modules/* when enablin

[Impala-ASF-CR] IMPALA-5586: Null-aware anti-join can take a long time to cancel

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7393 To un

[Impala-ASF-CR] IMPALA-5658: addtl. process/system-wide memory metrics

2017-07-24 Thread Tim Armstrong (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7472 to look at the new patch set (#5). Change subject: IMPALA-5658: addtl. process/system-wide memory metrics .. IMPA

[Impala-ASF-CR] IMPALA-5658: addtl. process/system-wide memory metrics

2017-07-24 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5658: addtl. process/system-wide memory metrics .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/7472/3/be/src/common/init.cc File be/src

[Impala-ASF-CR] IMPALA-5658: addtl. process/system-wide memory metrics

2017-07-24 Thread Tim Armstrong (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7472 to look at the new patch set (#4). Change subject: IMPALA-5658: addtl. process/system-wide memory metrics .. IMPA

[Impala-ASF-CR] IMPALA-5715: (potential mitigation) don't destroy MemTracker during Close()

2017-07-24 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/7492 Change subject: IMPALA-5715: (potential mitigation) don't destroy MemTracker during Close() .. IMPALA-5715: (potential mitigation) don't

[Impala-ASF-CR] IMPALA-4905: Don't send empty insert status to coordinator

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4905: Don't send empty insert status to coordinator .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7457 To unsubscr

[Impala-ASF-CR] IMPALA-4905: Don't send empty insert status to coordinator

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4905: Don't send empty insert status to coordinator .. IMPALA-4905: Don't send empty insert status to coordinator If the coordinator, in

[native-toolchain-CR] Add linker's version script for OpenSSL library.

2017-07-24 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: Add linker's version script for OpenSSL library. .. Patch Set 4: Filed IMPALA-5714 for some more background information. To answer your question, I intentionally s

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 9: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-24 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 9: Code-Review+2 Looks like another flaky test (Raised IMPALA-5711).

[Impala-ASF-CR] [DOCS] Take out not-production-ready notice from ADLS page

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: [DOCS] Take out not-production-ready notice from ADLS page .. [DOCS] Take out not-production-ready notice from ADLS page Change-Id: I127b7806fec

[Impala-ASF-CR] [DOCS] Take out not-production-ready notice from ADLS page

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Take out not-production-ready notice from ADLS page .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7487 To unsubscr

[Impala-ASF-CR] [DOCS] Take out not-production-ready notice from ADLS page

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Take out not-production-ready notice from ADLS page .. Patch Set 2: Build started: https://jenkins.impala.io/job/gerrit-docs-submit/146/ -- To v

[Impala-ASF-CR] IMPALA-4939, IMPALA-4939: Decimal V2 multiplication

2017-07-24 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#3). Change subject: IMPALA-4939, IMPALA-4939: Decimal V2 multiplication .. IMPALA-4939, IMPALA-4939: Decimal V2 multiplication Implement the new DECIMAL return type rules for mult

[Impala-ASF-CR] IMPALA-4939, IMPALA-4939: Decimal V2 multiplication

2017-07-24 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#3). Change subject: IMPALA-4939, IMPALA-4939: Decimal V2 multiplication .. IMPALA-4939, IMPALA-4939: Decimal V2 multiplication Implement the new DECIMAL return type rules for mult

[Impala-ASF-CR] IMPALA-4939, IMPALA-4939: Decimal V2 multiplication

2017-07-24 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4939, IMPALA-4939: Decimal V2 multiplication .. Patch Set 2: (11 comments) http://gerrit.cloudera.org:8080/#/c/7438/1/be/src/exprs/expr-test.cc File

[Impala-ASF-CR] IMPALA-4939, IMPALA-4939: Decimal V2 multiplication

2017-07-24 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#3). Change subject: IMPALA-4939, IMPALA-4939: Decimal V2 multiplication .. IMPALA-4939, IMPALA-4939: Decimal V2 multiplication Implement the new DECIMAL return type rules for mult

[native-toolchain-CR] Add linker's version script for OpenSSL library.

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Add linker's version script for OpenSSL library. .. Patch Set 4: > For question 2, I tried looking into how Ubuntu generated the > version script file but didn

[native-toolchain-CR] Add linker's version script for OpenSSL library.

2017-07-24 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: Add linker's version script for OpenSSL library. .. Patch Set 4: For question 2, I tried looking into how Ubuntu generated the version script file but didn't find

[Impala-ASF-CR] IMPALA-5709: Remove mini-impala-cluster

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5709: Remove mini-impala-cluster .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/928/ -- To view, visit http:/

[Impala-ASF-CR] IMPALA-5709: Remove mini-impala-cluster

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5709: Remove mini-impala-cluster .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7488 To unsubscribe, visit http://gerri

[Impala-ASF-CR] IMPALA-5658: addtl. process/system-wide memory metrics

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5658: addtl. process/system-wide memory metrics .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/7472/3/be/src/common/init.cc File be/sr

[Impala-ASF-CR] [DOCS] Take out not-production-ready notice from ADLS page

2017-07-24 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: [DOCS] Take out not-production-ready notice from ADLS page .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7487 To unsubscribe,

[Impala-ASF-CR] IMPALA-4795: Allow fetching function obj from catalog using signature

2017-07-24 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4795: Allow fetching function obj from catalog using signature .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7479/1/fe/src/main

[Impala-ASF-CR] Bump Kudu version to 27854fd

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: Bump Kudu version to 27854fd .. Bump Kudu version to 27854fd Change-Id: I0b9414c21faca00e4a64a35888bd50caac94318f Reviewed-on: http://gerrit.clo

[Impala-ASF-CR] Bump Kudu version to 27854fd

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Bump Kudu version to 27854fd .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7486 To unsubscribe, visit http://gerrit.cloud

[Impala-ASF-CR] IMPALA-4795: Allow fetching function obj from catalog using signature

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4795: Allow fetching function obj from catalog using signature .. Patch Set 1: (2 comments) Adding to Bharath's comments; please address those too. ht

[Impala-ASF-CR] IMPALA-5709: Remove mini-impala-cluster

2017-07-24 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/7488 Change subject: IMPALA-5709: Remove mini-impala-cluster .. IMPALA-5709: Remove mini-impala-cluster Remove untested / unused mini-impala

[Impala-ASF-CR] IMPALA-5586: Null-aware anti-join can take a long time to cancel

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel .. Patch Set 6: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/927/

[Impala-ASF-CR] IMPALA-5658: addtl. process/system-wide memory metrics

2017-07-24 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5658: addtl. process/system-wide memory metrics .. Patch Set 2: (7 comments) http://gerrit.cloudera.org:8080/#/c/7472/2/be/src/common/init.cc File be/src

[Impala-ASF-CR] IMPALA-5658: addtl. process/system-wide memory metrics

2017-07-24 Thread Tim Armstrong (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7472 to look at the new patch set (#3). Change subject: IMPALA-5658: addtl. process/system-wide memory metrics .. IMPA

[native-toolchain-CR] Add linker's version script for OpenSSL library.

2017-07-24 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: Add linker's version script for OpenSSL library. .. Patch Set 4: Answers to your questions: 1. I didn't try overriding LD_LIBRARY_PATH with other libraries in the

[Impala-ASF-CR] [DOCS] Take out not-production-ready notice from ADLS page

2017-07-24 Thread Greg Rahn (Code Review)
Greg Rahn has posted comments on this change. Change subject: [DOCS] Take out not-production-ready notice from ADLS page .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7487 To unsubscribe, visi

[Impala-ASF-CR] [DOCS] Take out not-production-ready notice from ADLS page

2017-07-24 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Take out not-production-ready notice from ADLS page .. Patch Set 1: (2 comments) If the reusable element is never referenced, it's dead code from the per

[Impala-ASF-CR] [DOCS] Take out not-production-ready notice from ADLS page

2017-07-24 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#2). Change subject: [DOCS] Take out not-production-ready notice from ADLS page .. [DOCS] Take out not-production-ready notice from ADLS page Change-Id: I127b7806feca810503ba3dd000a8e97

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 8: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-ver

[native-toolchain-CR] Add linker's version script for OpenSSL library.

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Add linker's version script for OpenSSL library. .. Patch Set 4: I think it'd be useful to have an IMPALA JIRA attached to this -- To view, visit http://gerri

[native-toolchain-CR] Add linker's version script for OpenSSL library.

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Add linker's version script for OpenSSL library. .. Patch Set 4: Thanks, Michael. More questions: 1) is this only a requirement for OpenSSL on Debian? Do you ha

[Impala-ASF-CR] IMPALA-4905: Don't send empty insert status to coordinator

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4905: Don't send empty insert status to coordinator .. Patch Set 2: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/926/ -- To

[native-toolchain-CR] Add linker's version script for OpenSSL library.

2017-07-24 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: Add linker's version script for OpenSSL library. .. Patch Set 4: Added the error message hit by statestored and friends without this change. In fact, if one were t

[native-toolchain-CR] Add linker's version script for OpenSSL library.

2017-07-24 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new patch set (#4). Change subject: Add linker's version script for OpenSSL library. .. Add linker's version script for OpenSSL library. Debian based distribution (e.g. Ubuntu) expects OpenSSL dynamical

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-24 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5511: Add process start time to debug web page .. Patch Set 9: (8 comments) http://gerrit.cloudera.org:8080/#/c/7363/9/be/src/statestore/statestore.cc F

[native-toolchain-CR] Add linker's version script for OpenSSL library.

2017-07-24 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new patch set (#3). Change subject: Add linker's version script for OpenSSL library. .. Add linker's version script for OpenSSL library. Debian based distribution (e.g. Ubuntu) expects OpenSSL dynamical

[Impala-ASF-CR] [DOCS] Take out not-production-ready notice from ADLS page

2017-07-24 Thread Greg Rahn (Code Review)
Greg Rahn has posted comments on this change. Change subject: [DOCS] Take out not-production-ready notice from ADLS page .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7487/1/docs/shared/impala_common.xml File

[Impala-ASF-CR] [DOCS] Update list of reserved words

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Update list of reserved words .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7452 To unsubscribe, visit http://gerr

[Impala-ASF-CR] [DOCS] Update list of reserved words

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: [DOCS] Update list of reserved words .. [DOCS] Update list of reserved words For 2.9, I believe the only new reserved keyword is TABLESAMPLE fro

[Impala-ASF-CR] [DOCS] Fix URL of link to glog project

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: [DOCS] Fix URL of link to glog project .. [DOCS] Fix URL of link to glog project Formerly the project used SVN and the instructions were posted

[Impala-ASF-CR] [DOCS] Fix URL of link to glog project

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Fix URL of link to glog project .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7447 To unsubscribe, visit http://ge

[Impala-ASF-CR] [DOCS] Take out not-production-ready notice from ADLS page

2017-07-24 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: [DOCS] Take out not-production-ready notice from ADLS page .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7487/1/docs/shared/impala_common.xml

[Impala-ASF-CR] [DOCS] Update list of reserved words

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Update list of reserved words .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-docs-submit/145/ -- To view, visit http://gerr

[Impala-ASF-CR] [DOCS] Fix URL of link to glog project

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Fix URL of link to glog project .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-docs-submit/144/ -- To view, visit http://ge

[Impala-ASF-CR] [DOCS] Take out not-production-ready notice from ADLS page

2017-07-24 Thread John Russell (Code Review)
John Russell has uploaded a new change for review. http://gerrit.cloudera.org:8080/7487 Change subject: [DOCS] Take out not-production-ready notice from ADLS page .. [DOCS] Take out not-production-ready notice from ADLS page

[Impala-ASF-CR] Bump Kudu version to 27854fd

2017-07-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 27854fd .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7486 To unsubscribe, visit http://gerrit.c

[Impala-ASF-CR] Bump Kudu version to 27854fd

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Bump Kudu version to 27854fd .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/925/ -- To view, visit http://gerrit.clo

[Impala-ASF-CR] Bump Kudu version to 27854fd

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/7486 Change subject: Bump Kudu version to 27854fd .. Bump Kudu version to 27854fd Change-Id: I0b9414c21faca00e4a64a35888bd50caac94318f --- M

[native-toolchain-CR] Remove Kudu build workaround for llvm stdlib path

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: Remove Kudu build workaround for llvm stdlib path .. Remove Kudu build workaround for llvm stdlib path We previously had to work around a Kudu build li

[native-toolchain-CR] Remove Kudu build workaround for llvm stdlib path

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Remove Kudu build workaround for llvm stdlib path .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7485 To unsubscribe, visit http:

[native-toolchain-CR] Remove Kudu build workaround for llvm stdlib path

2017-07-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Remove Kudu build workaround for llvm stdlib path .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7485 To unsubscribe,

[native-toolchain-CR] Remove Kudu build workaround for llvm stdlib path

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Remove Kudu build workaround for llvm stdlib path .. Patch Set 1: ran a full toolchain build, works as expected -- To view, visit http://gerrit.cloudera.org:8

[native-toolchain-CR] Remove Kudu build workaround for llvm stdlib path

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/7485 Change subject: Remove Kudu build workaround for llvm stdlib path .. Remove Kudu build workaround for llvm stdlib path We previously ha

[native-toolchain-CR] Add linker's version script for OpenSSL library.

2017-07-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Add linker's version script for OpenSSL library. .. Patch Set 2: (1 comment) what was happening when this wasn't included? can you add a link to show where th

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 8: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-24 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 8: Code-Review+2 Known flaky issue, giving it another shot. -- To v

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-24 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has posted comments on this change. Change subject: IMPALA-5511: Add process start time to debug web page .. Patch Set 8: (7 comments) http://gerrit.cloudera.org:8080/#/c/7363/8/be/src/catalog/catalog-server.cc Fi

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-24 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has uploaded a new patch set (#9). Change subject: IMPALA-5511: Add process start time to debug web page .. IMPALA-5511: Add process start time to debug web page Read the start date and time of the impalad, catalogd

[Impala-ASF-CR] IMPALA-5532: Stack-allocate compressors in RowBatch (de)serialization

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5532: Stack-allocate compressors in RowBatch (de)serialization .. IMPALA-5532: Stack-allocate compressors in RowBatch (de)serialization

[Impala-ASF-CR] IMPALA-5532: Stack-allocate compressors in RowBatch (de)serialization

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5532: Stack-allocate compressors in RowBatch (de)serialization .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7478

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 7: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-ver

[Impala-ASF-CR] IMPALA-4905: Don't send empty insert status to coordinator

2017-07-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4905: Don't send empty insert status to coordinator .. Patch Set 2: Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/921/ -- To