[Impala-ASF-CR] IMPALA-5259: Add REFRESH FUNCTIONS statement

2017-05-23 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#10). Change subject: IMPALA-5259: Add REFRESH FUNCTIONS statement .. IMPALA-5259: Add REFRESH FUNCTIONS statement Before this patch, Impala relied on INVALIDATE METADATA to load

[Impala-ASF-CR] IMPALA-5259: Add REFRESH FUNCTIONS statement

2017-05-23 Thread Taras Bobrovytsky (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6878 to look at the new patch set (#10). Change subject: IMPALA-5259: Add REFRESH FUNCTIONS statement .. IMPALA-5259: Add

[Impala-ASF-CR] IMPALA-5259: Add REFRESH FUNCTIONS statement

2017-05-23 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-5259: Add REFRESH FUNCTIONS statement .. Patch Set 9: (5 comments) http://gerrit.cloudera.org:8080/#/c/6878/9/fe/src/main/java/org/apache/impala/cat

[Impala-ASF-CR] IMPALA-4848: Add WIDTH BUCKET() function

2017-05-23 Thread anujphadke (Code Review)
anujphadke has posted comments on this change. Change subject: IMPALA-4848: Add WIDTH_BUCKET() function .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6023/4/be/src/exprs/math-functions-ir.cc File be/src/exprs

[Impala-ASF-CR] IMPALA-5030: Adds support for NVL2() function

2017-05-23 Thread Vincent Tran (Code Review)
Vincent Tran has uploaded a new patch set (#2). Change subject: IMPALA-5030: Adds support for NVL2() function .. IMPALA-5030: Adds support for NVL2() function This change adds support to the function NVL2(expr1, expr2, expr3). T

[Impala-ASF-CR] IMPALA-5030: Adds support for NVL2() function

2017-05-23 Thread Vincent Tran (Code Review)
Vincent Tran has abandoned this change. Change subject: IMPALA-5030: Adds support for NVL2() function .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/6972 To unsubscribe, visit http://gerrit.cloudera.org:8080/se

[Impala-ASF-CR] IMPALA-5030: Adds support for NVL2() function

2017-05-23 Thread Vincent Tran (Code Review)
Vincent Tran has uploaded a new change for review. http://gerrit.cloudera.org:8080/6972 Change subject: IMPALA-5030: Adds support for NVL2() function .. IMPALA-5030: Adds support for NVL2() function This change adds support t

[Impala-ASF-CR] IMPALA-5259: Add REFRESH FUNCTIONS statement

2017-05-23 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-5259: Add REFRESH FUNCTIONS statement .. Patch Set 9: (4 comments) http://gerrit.cloudera.org:8080/#/c/6878/9/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-5342: Add comments of loaded tables in the response of GetTables

2017-05-23 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-5342: Add comments of loaded tables in the response of GetTables .. Patch Set 3: Code-Review+2 Rebase, keep Alex's +2 -- To view, visit http://

[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created (BE)

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created (BE) .. Patch Set 7: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryr

[Impala-ASF-CR] IMPALA-4890/5143: Coordinator race involving TearDown()

2017-05-23 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4890/5143: Coordinator race involving TearDown() .. Patch Set 1: (2 comments) Where does ReleaseResources() get called for DML queries? It used to be in

[Impala-ASF-CR] IMPALA-1972/IMPALA-3882: Fix client request state map lock contention

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-1972/IMPALA-3882: Fix client_request_state_map_lock_ contention .. IMPALA-1972/IMPALA-3882: Fix client_request_state_map_lock_ contention

[Impala-ASF-CR] IMPALA-1972/IMPALA-3882: Fix client request state map lock contention

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-1972/IMPALA-3882: Fix client_request_state_map_lock_ contention .. Patch Set 11: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/67

[Impala-ASF-CR] IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs.

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs. .. IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs. Syntax: TABLESAMPLE SYST

[Impala-ASF-CR] IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs.

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs. .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6868 To unsubscri

[Impala-ASF-CR] IMPALA-5324: Fix version check in EvalDictionaryFilters

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5324: Fix version check in EvalDictionaryFilters .. Patch Set 1: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/6

[Impala-ASF-CR] IMPALA-4192: Disentangle Expr and ExprContext

2017-05-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4192: Disentangle Expr and ExprContext .. Patch Set 14: (70 comments) http://gerrit.cloudera.org:8080/#/c/5483/14/be/src/exec/aggregation-node.cc File be/sr

[Impala-ASF-CR] IMPALA-4192: Disentangle Expr and ExprContext

2017-05-23 Thread Michael Ho (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5483 to look at the new patch set (#15). Change subject: IMPALA-4192: Disentangle Expr and ExprContext .. IMPALA-4192:

[Impala-ASF-CR] IMPALA-5325: Do not update totalHdfsBytes /numHdfsFiles on Catalogd

2017-05-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5325: Do not update totalHdfsBytes_/numHdfsFiles_ on Catalogd .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6970/1/fe/src/main/java/org/a

[native-toolchain-CR] Bumped Kudu version to 795c435.

2017-05-23 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Bumped Kudu version to 795c435. .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6971 To unsubscribe, visit http://gerrit.cloude

[native-toolchain-CR] Bumped Kudu version to 795c435.

2017-05-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6971 Change subject: Bumped Kudu version to 795c435. .. Bumped Kudu version to 795c435. This is needed for IMPALA-5167. Change-Id:

[native-toolchain-CR] Bumped Kudu version to 795c435.

2017-05-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bumped Kudu version to 795c435. .. Patch Set 1: http://unittest.jenkins.cloudera.com/job/verify-impala-toolchain-package-build/393/ -- To view, visit

[Impala-ASF-CR] IMPALA-5325: Do not update totalHdfsBytes /numHdfsFiles on Catalogd

2017-05-23 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new change for review. http://gerrit.cloudera.org:8080/6970 Change subject: IMPALA-5325: Do not update totalHdfsBytes_/numHdfsFiles_ on Catalogd .. IMPALA-5325: Do not update totalHdfsBytes

[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created (BE)

2017-05-23 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created (BE) .. Patch Set 7: Code-Review+2 rebase -- To view, visit http://gerrit.cloudera.org:8080/6792 To uns

[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created (BE)

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created (BE) .. Patch Set 7: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/612/ -

[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created (BE)

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created (BE) .. Patch Set 6: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryr

[Impala-ASF-CR] IMPALA-5157: [DOCS] Remove sortby() hint from new features

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5157: [DOCS] Remove sortby() hint from new features .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6965 To unsubscr

[Impala-ASF-CR] IMPALA-5157: [DOCS] Remove sortby() hint from new features

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5157: [DOCS] Remove sortby() hint from new features .. IMPALA-5157: [DOCS] Remove sortby() hint from new features The writeup for sortby(

[Impala-ASF-CR] IMPALA-5157: [DOCS] Remove sortby() hint from new features

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5157: [DOCS] Remove sortby() hint from new features .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/118/ --

[Impala-ASF-CR] IMPALA-4890/5143: Coordinator race involving TearDown()

2017-05-23 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4890/5143: Coordinator race involving TearDown() .. Patch Set 1: (1 comment) Patch looks pretty reasonable, will finish shortly. http://gerrit.cloudera

[Impala-ASF-CR] IMPALA-5342: Add comments of loaded tables in the response of GetTables

2017-05-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5342: Add comments of loaded tables in the response of GetTables .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6933 To uns

[Impala-ASF-CR] IMPALA-1972/IMPALA-3882: Fix client request state map lock contention

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-1972/IMPALA-3882: Fix client_request_state_map_lock_ contention .. Patch Set 11: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-d

[Impala-ASF-CR] IMPALA-1972/IMPALA-3882: Fix client request state map lock contention

2017-05-23 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-1972/IMPALA-3882: Fix client_request_state_map_lock_ contention .. Patch Set 11: Code-Review+2 Carrying +2. -- To view, visit http://gerrit.clou

[Impala-ASF-CR] IMPALA-1972/IMPALA-3882: Fix client request state map lock contention

2017-05-23 Thread Bharath Vissapragada (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6707 to look at the new patch set (#11). Change subject: IMPALA-1972/IMPALA-3882: Fix client_request_state_map_lock_ contention ...

[Impala-ASF-CR] IMPALA-5157: [DOCS] Remove sortby() hint from new features

2017-05-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5157: [DOCS] Remove sortby() hint from new features .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6965 To unsubscribe, visi

[Impala-ASF-CR] IMPALA-1972/IMPALA-3882: Fix client request state map lock contention

2017-05-23 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-1972/IMPALA-3882: Fix client_request_state_map_lock_ contention .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/6707/10/tests/cu

[Impala-ASF-CR] IMPALA-4956: [DOCS] Correct EXPLAIN LEVEL mnemonics and numbers

2017-05-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4956: [DOCS] Correct EXPLAIN_LEVEL mnemonics and numbers .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6966/1/docs/topics/impala_explain_pl

[Impala-ASF-CR] IMPALA-5324: Fix version check in EvalDictionaryFilters

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5324: Fix version check in EvalDictionaryFilters .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/610/ -- T

[Impala-ASF-CR] IMPALA-5324: Fix version check in EvalDictionaryFilters

2017-05-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5324: Fix version check in EvalDictionaryFilters .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6969 To unsubscribe, visit h

[Impala-ASF-CR] IMPALA-5324: Fix version check in EvalDictionaryFilters

2017-05-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded a new change for review. http://gerrit.cloudera.org:8080/6969 Change subject: IMPALA-5324: Fix version check in EvalDictionaryFilters .. IMPALA-5324: Fix version check in EvalDictionaryFilters Due t

[Impala-ASF-CR] IMPALA-4803: [DOCS] 2.9 release notes placeholder

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4803: [DOCS] 2.9 release notes placeholder .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6964 To unsubscribe, visi

[Impala-ASF-CR] IMPALA-4803: [DOCS] 2.9 release notes placeholder

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4803: [DOCS] 2.9 release notes placeholder .. IMPALA-4803: [DOCS] 2.9 release notes placeholder Change-Id: I636a6f2dcd0555ab9b46304e3a729

[Impala-ASF-CR] IMPALA-5030: Add support to NVL2() function

2017-05-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5030: Add support to NVL2() function .. Patch Set 1: (10 comments) I left my CR comments for educational purposes, although most of them will be mute when s

[Impala-ASF-CR] IMPALA-4803: [DOCS] 2.9 release notes placeholder

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4803: [DOCS] 2.9 release notes placeholder .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/117/ -- To view,

[Impala-ASF-CR] IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs.

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs. .. Patch Set 7: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/609/ --

[Impala-ASF-CR] IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs.

2017-05-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs. .. Patch Set 7: Code-Review+2 Minor correction of expected result in ToSqlTest due to an improvement in pr

[Impala-ASF-CR] IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs.

2017-05-23 Thread Alex Behm (Code Review)
Hello Marcel Kornacker, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6868 to look at the new patch set (#7). Change subject: IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs.

[Impala-ASF-CR] IMPALA-5333: Add support for Impala to work with ADLS

2017-05-23 Thread Sailesh Mukil (Code Review)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6910 to look at the new patch set (#4). Change subject: IMPALA-5333: Add support for Impala to work with ADLS .. IMPAL

[Impala-ASF-CR] IMPALA-5333: Add support for Impala to work with ADLS

2017-05-23 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5333: Add support for Impala to work with ADLS .. Patch Set 3: > > I have a few high level questions about this patch. This patch > > treats S3 and ADL t

[Impala-ASF-CR] IMPALA-4956: [DOCS] Correct EXPLAIN LEVEL mnemonics and numbers

2017-05-23 Thread John Russell (Code Review)
John Russell has uploaded a new change for review. http://gerrit.cloudera.org:8080/6966 Change subject: IMPALA-4956: [DOCS] Correct EXPLAIN_LEVEL mnemonics and numbers .. IMPALA-4956: [DOCS] Correct EXPLAIN_LEVEL mnemonics and

[Impala-ASF-CR] IMPALA-5157: [DOCS] Remove sortby() hint from new features

2017-05-23 Thread John Russell (Code Review)
John Russell has uploaded a new change for review. http://gerrit.cloudera.org:8080/6965 Change subject: IMPALA-5157: [DOCS] Remove sortby() hint from new features .. IMPALA-5157: [DOCS] Remove sortby() hint from new features

[Impala-ASF-CR] IMPALA-5164: Fix flaky benchmarks

2017-05-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5164: Fix flaky benchmarks .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/6935/2/be/src/util/benchmark.cc File be/src/util/benchmark.cc

[Impala-ASF-CR] Loads all TPC-DS tables

2017-05-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: Loads all TPC-DS tables .. Patch Set 5: Code-Review+1 Carry +1. -- To view, visit http://gerrit.cloudera.org:8080/6877 To unsubscribe, visit http://gerrit.clouder

[Impala-ASF-CR] Loads all TPC-DS tables

2017-05-23 Thread Michael Ho (Code Review)
Hello Mostafa Mokhtar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6877 to look at the new patch set (#5). Change subject: Loads all TPC-DS tables .. Loads all TPC-DS tables This chan

[Impala-ASF-CR] Loads all TPC-DS tables

2017-05-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: Loads all TPC-DS tables .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/6877/3/testdata/workloads/functional-query/queries/QueryTest/seq-writer.te

[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created (BE)

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created (BE) .. Patch Set 6: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/608/ -

[Impala-ASF-CR] IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs.

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs. .. Patch Set 6: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun

[Impala-ASF-CR] IMPALA-4164: Use inline hints instead of always-inline

2017-05-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4164: Use inline hints instead of always-inline .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/6941/2/be/src/codegen/llvm-codegen.cc File

[Impala-ASF-CR] IMPALA-4866: Hash join node does not apply limits correctly

2017-05-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4866: Hash join node does not apply limits correctly .. Patch Set 2: (6 comments) Looking pretty good I think. http://gerrit.cloudera.org:8080/#/c/6778/

[Impala-ASF-CR] IMPALA-5259: Add REFRESH FUNCTIONS statement

2017-05-23 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#9). Change subject: IMPALA-5259: Add REFRESH FUNCTIONS statement .. IMPALA-5259: Add REFRESH FUNCTIONS statement Before this patch, Impala relied on INVALIDATE METADATA to load

[Impala-ASF-CR] IMPALA-5259: Add REFRESH FUNCTIONS statement

2017-05-23 Thread Taras Bobrovytsky (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6878 to look at the new patch set (#9). Change subject: IMPALA-5259: Add REFRESH FUNCTIONS statement .. IMPALA-5259: Add R

[Impala-ASF-CR] IMPALA-5259: Add REFRESH FUNCTIONS statement

2017-05-23 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-5259: Add REFRESH FUNCTIONS statement .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/6878/8/fe/src/main/java/org/apache/impala/cata

[Impala-ASF-CR] Loads all TPC-DS tables

2017-05-23 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. Change subject: Loads all TPC-DS tables .. Patch Set 4: -Code-Review -- To view, visit http://gerrit.cloudera.org:8080/6877 To unsubscribe, visit http://gerrit.cloudera.org:8080/set

[Impala-ASF-CR] Loads all TPC-DS tables

2017-05-23 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. Change subject: Loads all TPC-DS tables .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6877 To unsubscribe, visit http://gerrit.cloudera.org:8080/se

[Impala-ASF-CR] IMPALA-4164: Use inline hints instead of always-inline

2017-05-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4164: Use inline hints instead of always-inline .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6941/2/be/src/codegen/llvm-codegen.cc Fil

[Impala-ASF-CR] IMPALA-5347: Parquet scanner microoptimizations

2017-05-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5347: Parquet scanner microoptimizations .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6950/5/be/src/exec/parquet-column-readers.cc Fil

[Impala-ASF-CR] Loads all TPC-DS tables

2017-05-23 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. Change subject: Loads all TPC-DS tables .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6877/3/tests/query_test/test_mem_usage_scaling.py File tests/query_test/test_

[Impala-ASF-CR] IMPALA-5347: Parquet scanner microoptimizations

2017-05-23 Thread Tim Armstrong (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6950 to look at the new patch set (#8). Change subject: IMPALA-5347: Parquet scanner microoptimizations .. IMPALA-5347: Par

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-05-23 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#8). Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. IMPALA-4864 Speed up single slot predicates with dictionaries When dictionaries are present we can p

[Impala-ASF-CR] IMPALA-4803: [DOCS] 2.9 release notes placeholder

2017-05-23 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4803: [DOCS] 2.9 release notes placeholder .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6964 To unsubscribe, visit htt

[Impala-ASF-CR] PREVIEW: IMPALA-5160: adjust spill buffer size based on planner estimates

2017-05-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#3). Change subject: PREVIEW: IMPALA-5160: adjust spill buffer size based on planner estimates .. PREVIEW: IMPALA-5160: adjust spill buffer size based on planner estimates Posting for

[Impala-ASF-CR] PREVIEW: IMPALA-5160: adjust spill buffer size based on planner estimates

2017-05-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). Change subject: PREVIEW: IMPALA-5160: adjust spill buffer size based on planner estimates .. PREVIEW: IMPALA-5160: adjust spill buffer size based on planner estimates Posting for

[Impala-ASF-CR] IMPALA-4803: [DOCS] 2.9 release notes placeholder

2017-05-23 Thread John Russell (Code Review)
John Russell has uploaded a new change for review. http://gerrit.cloudera.org:8080/6964 Change subject: IMPALA-4803: [DOCS] 2.9 release notes placeholder .. IMPALA-4803: [DOCS] 2.9 release notes placeholder Change-Id: I636a6f

[Impala-ASF-CR] Loads all TPC-DS tables

2017-05-23 Thread Mostafa Mokhtar (Code Review)
Mostafa Mokhtar has posted comments on this change. Change subject: Loads all TPC-DS tables .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6877/3/testdata/workloads/functional-query/queries/QueryTest/seq-write

[Impala-ASF-CR] IMPALA-5031: Initialize only parsing header before reading it

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5031: Initialize only_parsing_header_ before reading it .. IMPALA-5031: Initialize only_parsing_header_ before reading it UBSan reports "

[Impala-ASF-CR] IMPALA-5031: Initialize only parsing header before reading it

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5031: Initialize only_parsing_header_ before reading it .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6937 To unsu

[Impala-ASF-CR] Loads all TPC-DS tables

2017-05-23 Thread Michael Ho (Code Review)
Hello Mostafa Mokhtar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6877 to look at the new patch set (#4). Change subject: Loads all TPC-DS tables .. Loads all TPC-DS tables This chan

[Impala-ASF-CR] Loads all TPC-DS tables

2017-05-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: Loads all TPC-DS tables .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/6877/3/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java

[Impala-ASF-CR] IMPALA-4803: [DOCS] 2.9 release notes placeholder

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4803: [DOCS] 2.9 release notes placeholder .. IMPALA-4803: [DOCS] 2.9 release notes placeholder Start with placeholder for 2.9 new featur

[Impala-ASF-CR] IMPALA-4803: [DOCS] 2.9 release notes placeholder

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4803: [DOCS] 2.9 release notes placeholder .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6954 To unsubscribe, visi

[Impala-ASF-CR] [DOCS] Add several items to "known issues" page

2017-05-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: [DOCS] Add several items to "known issues" page .. Patch Set 3: Code-Review+1 IMPALA-4432 looks good. Please let me know if all other JIRA are resolved and I can +

[Impala-ASF-CR] IMPALA-4803: [DOCS] 2.9 release notes placeholder

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4803: [DOCS] 2.9 release notes placeholder .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/116/ -- To view,

[Impala-ASF-CR] IMPALA-4623: Enable file handle cache

2017-05-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded a new patch set (#12). Change subject: IMPALA-4623: Enable file handle cache .. IMPALA-4623: Enable file handle cache Currently, every scan range maintains a file handle, even when multiple scan ranges

[Impala-ASF-CR] IMPALA-4623: Enable file handle cache

2017-05-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. Change subject: IMPALA-4623: Enable file handle cache .. Patch Set 11: (4 comments) http://gerrit.cloudera.org:8080/#/c/6478/11/be/src/runtime/disk-io-mgr-scan-range.cc File be

[Impala-ASF-CR] IMPALA-4803: [DOCS] 2.9 release notes placeholder

2017-05-23 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4803: [DOCS] 2.9 release notes placeholder .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/6954/1//COMMIT_MSG Commit Messag

[Impala-ASF-CR] IMPALA-4803: [DOCS] 2.9 release notes placeholder

2017-05-23 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#2). Change subject: IMPALA-4803: [DOCS] 2.9 release notes placeholder .. IMPALA-4803: [DOCS] 2.9 release notes placeholder Start with placeholder for 2.9 new features topic. Initially

[Impala-ASF-CR] IMPALA-4803: [DOCS] 2.9 release notes

2017-05-23 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: IMPALA-4803: [DOCS] 2.9 release notes .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6954/1//COMMIT_MSG Commit Message: PS1, Line 7: IMPALA-480

[Impala-ASF-CR] IMPALA-5347: Parquet scanner microoptimizations

2017-05-23 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5347: Parquet scanner microoptimizations .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6950/5/be/src/exec/parquet-column-readers.cc File be

[Impala-ASF-CR] IMPALA-5347: Parquet scanner microoptimizations

2017-05-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5347: Parquet scanner microoptimizations .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6950/5/be/src/exec/parquet-column-readers.cc Fil

[Impala-ASF-CR] IMPALA-5347: Parquet scanner microoptimizations

2017-05-23 Thread Tim Armstrong (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6950 to look at the new patch set (#7). Change subject: IMPALA-5347: Parquet scanner microoptimizations .. IMPALA-5347: Par

[Impala-ASF-CR] IMPALA-5347: Parquet scanner microoptimizations

2017-05-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5347: Parquet scanner microoptimizations .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6950/5/be/src/exec/parquet-column-readers.cc Fil

[Impala-ASF-CR] IMPALA-5347: Parquet scanner microoptimizations

2017-05-23 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5347: Parquet scanner microoptimizations .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/6950/5/be/src/exec/parquet-column-readers.cc File b

[Impala-ASF-CR] IMPALA-5347: Parquet scanner microoptimizations

2017-05-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5347: Parquet scanner microoptimizations .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/6950/5/be/src/exec/parquet-column-readers.cc Fi

[Impala-ASF-CR] IMPALA-5347: Parquet scanner microoptimizations

2017-05-23 Thread Tim Armstrong (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6950 to look at the new patch set (#6). Change subject: IMPALA-5347: Parquet scanner microoptimizations .. IMPALA-5347: Par

[Impala-ASF-CR] IMPALA-1972/IMPALA-3882: Fix client request state map lock contention

2017-05-23 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-1972/IMPALA-3882: Fix client_request_state_map_lock_ contention .. Patch Set 10: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/6707/10/tests

[Impala-ASF-CR] Loads all TPC-DS tables

2017-05-23 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. Change subject: Loads all TPC-DS tables .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/6877/3/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java Fi

[Impala-ASF-CR] IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs.

2017-05-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs. .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6868 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs.

2017-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs. .. Patch Set 6: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/607/ --

[Impala-ASF-CR] IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs.

2017-05-23 Thread Alex Behm (Code Review)
Hello Marcel Kornacker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6868 to look at the new patch set (#6). Change subject: IMPALA-5309: Adds TABLESAMPLE clause for HDFS table refs. ..

[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created (BE)

2017-05-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created (BE) .. Patch Set 6: > anything holding this up? if not, can you submit? working on getting a to

  1   2   >