[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-02-02 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/5776/10//COMMIT_MSG Commit Message: Line 13: Change-Id: I1780a7d8

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-02-02 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/5776/10/be/src/exprs/string-functions-ir.cc File be/src/exprs/st

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-02-02 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/5776/10/be/src/exprs/expr-test.cc File be/src/exprs/expr-test.cc

[Impala-ASF-CR] IMPALA-4675 Lowercase parquet column names prior to matching

2017-02-02 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4675 Lowercase parquet column names prior to matching .. Patch Set 2: (2 comments) Thanks for taking this on! Will you be able to add test cases? Please rea

[Impala-ASF-CR] IMPALA-4876: Remove test suffix from test names

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4876: Remove _test suffix from test names .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5886 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4876: Remove test suffix from test names

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4876: Remove _test suffix from test names .. IMPALA-4876: Remove _test suffix from test names After IMPALA-4735 has been fixed, this suff

[Impala-ASF-CR] IMPALA-4675 Lowercase parquet column names prior to matching

2017-02-02 Thread Nathan Salmon (Code Review)
Nathan Salmon has uploaded a new patch set (#2). Change subject: IMPALA-4675 Lowercase parquet column names prior to matching .. IMPALA-4675 Lowercase parquet column names prior to matching With the introduction of the query opt

[Impala-ASF-CR] IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables .. IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables

[Impala-ASF-CR] IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen

2017-02-02 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen .. Patch Set 5: (6 comments) http://gerrit.cloudera.org:8080/#/c/5732/4/be/src/codegen/llvm-

[Impala-ASF-CR] IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5815

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-02-02 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#10). Change subject: IMPALA-4729: Implement REPLACE() .. IMPALA-4729: Implement REPLACE() This turned out to be slightly non-trivial as REPLACE is already a keyword, and thus the parser

[Impala-ASF-CR] IMPALA-4810: Add DECIMAL V2 query option

2017-02-02 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4810: Add DECIMAL_V2 query option .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5889/1//COMMIT_MSG Commit Message: Line 7: IMPALA-4810: Ad

[Impala-ASF-CR] IMPALA-4810: Add DECIMAL V2 query option

2017-02-02 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4810: Add DECIMAL_V2 query option .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/5889/1//COMMIT_MSG Commit Message: Line 7: IMPALA-4810:

[Impala-ASF-CR] IMPALA-4810: Add DECIMAL V2 query option

2017-02-02 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/5889 Change subject: IMPALA-4810: Add DECIMAL_V2 query option .. IMPALA-4810: Add DECIMAL_V2 query option Change-Id: I6cf25fe5c1766c86ebfe196d4

[Impala-ASF-CR] IMPALA-4868: Fix flaky TestRequestPoolService.testUpdatingConfigs

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4868: Fix flaky TestRequestPoolService.testUpdatingConfigs .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5876 To

[Impala-ASF-CR] IMPALA-4868: Fix flaky TestRequestPoolService.testUpdatingConfigs

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4868: Fix flaky TestRequestPoolService.testUpdatingConfigs .. IMPALA-4868: Fix flaky TestRequestPoolService.testUpdatingConfigs Occasion

[Impala-ASF-CR] IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen

2017-02-02 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen .. Patch Set 4: (9 comments) http://gerrit.cloudera.org:8080/#/c/5732/4//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen

2017-02-02 Thread Michael Ho (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5732 to look at the new patch set (#5). Change subject: IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen

[Impala-ASF-CR] IMPALA-4738: STDDEV SAMP should return NULL for single record input

2017-02-02 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4738: STDDEV_SAMP should return NULL for single record input .. Patch Set 3: Code-Review+2 (2 comments) Thanks! Please let John know that the existing

[Impala-ASF-CR] IMPALA-1427: Improvements to "Unknown disk-ID" warning

2017-02-02 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-1427: Improvements to "Unknown disk-ID" warning .. Patch Set 4: (6 comments) So much better! http://gerrit.cloudera.org:8080/#/c/5828/4/be/src/exec/hdfs-sca

[Impala-ASF-CR] IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC

2017-02-02 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/572

[Impala-ASF-CR] IMPALA-4738: STDDEV SAMP should return NULL for single record input

2017-02-02 Thread anujphadke (Code Review)
anujphadke has posted comments on this change. Change subject: IMPALA-4738: STDDEV_SAMP should return NULL for single record input .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/5800/2/testdata/workloads/fun

[Impala-ASF-CR] IMPALA-4738: STDDEV SAMP should return NULL for single record input

2017-02-02 Thread anujphadke (Code Review)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5800 to look at the new patch set (#3). Change subject: IMPALA-4738: STDDEV_SAMP should return NULL for single record input ..

[Impala-ASF-CR] IMPALA-4876: Remove test suffix from test names

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4876: Remove _test suffix from test names .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/232/ -- To view,

[Impala-ASF-CR] IMPALA-4876: Remove test suffix from test names

2017-02-02 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4876: Remove _test suffix from test names .. Patch Set 1: Yes, that would be great, if you could merge this for me. Thanks for the help. -- To view, visit

[Impala-ASF-CR] IMPALA-4874: Increase the maximum KRPC message size to 4GB

2017-02-02 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/5887 Change subject: IMPALA-4874: Increase the maximum KRPC message size to 4GB .. IMPALA-4874: Increase the maximum KRPC message size to 4GB

[Impala-ASF-CR] IMPALA-4856: Port ImpalaInternalService to KRPC

2017-02-02 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/5888 Change subject: IMPALA-4856: Port ImpalaInternalService to KRPC .. IMPALA-4856: Port ImpalaInternalService to KRPC This patch ports the

[Impala-ASF-CR] IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool

2017-02-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#5). Change subject: IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool .. IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool Add a copy of BufferedTupleStream that a

[Impala-ASF-CR] IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool

2017-02-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool .. Patch Set 4: (37 comments) http://gerrit.cloudera.org:8080/#/c/5811/4/be/src/runtime/buffered-tup

[Impala-ASF-CR] IMPALA-4876: Remove test suffix from test names

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4876: Remove _test suffix from test names .. Patch Set 1: Do you want me to start the pre-merge Jenkins test? -- To view, visit http://gerrit.cloudera.org:8

[Impala-ASF-CR] IMPALA-4876: Remove test suffix from test names

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4876: Remove _test suffix from test names .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5886 To unsubscribe, visit http://g

[Impala-ASF-CR] IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables .. Patch Set 4: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dry

[Impala-ASF-CR] IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables

2017-02-02 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5

[Impala-ASF-CR] IMPALA-4876: Remove test suffix from test names

2017-02-02 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/5886 Change subject: IMPALA-4876: Remove _test suffix from test names .. IMPALA-4876: Remove _test suffix from test names After IMPALA-4735 has

[Impala-ASF-CR] IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4792: Fix number of distinct values for a CASE with constant outputs .. Patch Set 8: Verified+1 -- To view, visit http://gerrit.cloudera.org:808

[Impala-ASF-CR] IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4792: Fix number of distinct values for a CASE with constant outputs .. IMPALA-4792: Fix number of distinct values for a CASE with consta

[Impala-ASF-CR] [DOCS] Fix typo in link text

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Fix typo in link text .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5881 To unsubscribe, visit http://gerrit.cloud

[Impala-ASF-CR] [DOCS] Fix typo in link text

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: [DOCS] Fix typo in link text .. [DOCS] Fix typo in link text The "Known Issue" link to IMPALA-1480 is the full URL + the text "IMPALA-1480" afte

[Impala-ASF-CR] [DOCS] Fix typo in link text

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: [DOCS] Fix typo in link text .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5881 To unsubscribe, visit http://gerrit.cloudera.org:8

[Impala-ASF-CR] [DOCS] Fix typo in link text

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Fix typo in link text .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/34/ -- To view, visit http://gerrit.cl

[Impala-ASF-CR] IMPALA-4868: Fix flaky TestRequestPoolService.testUpdatingConfigs

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4868: Fix flaky TestRequestPoolService.testUpdatingConfigs .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/

[Impala-ASF-CR] [DOCS] Fix typo in link text

2017-02-02 Thread John Russell (Code Review)
Hello Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5881 to look at the new patch set (#2). Change subject: [DOCS] Fix typo in link text .. [DOCS] Fix typo in link text The "

[Impala-ASF-CR] IMPALA-1427: Improvements to "Unknown disk-ID" warning

2017-02-02 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-1427: Improvements to "Unknown disk-ID" warning .. Patch Set 4: A sample output looks like this. [localhost:21000] > explain select * from logging

[Impala-ASF-CR] IMPALA-1427: Improvements to "Unknown disk-ID" warning

2017-02-02 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#4). Change subject: IMPALA-1427: Improvements to "Unknown disk-ID" warning .. IMPALA-1427: Improvements to "Unknown disk-ID" warning This commit, - Removes the runtime unknown

[Impala-ASF-CR] [DOCS] Fix typo in link text

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: [DOCS] Fix typo in link text .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5881 To unsubscribe, visit http://gerrit.cloudera.org:8

[Impala-ASF-CR] [DOCS] Fix typo in link text

2017-02-02 Thread John Russell (Code Review)
John Russell has uploaded a new change for review. http://gerrit.cloudera.org:8080/5881 Change subject: [DOCS] Fix typo in link text .. [DOCS] Fix typo in link text The "Known Issue" link to IMPALA-1480 is the full URL + the

[Impala-ASF-CR] IMPALA-1427: Improvements to "Unknown disk-ID" warning

2017-02-02 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-1427: Improvements to "Unknown disk-ID" warning .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/5828/3/be/src/exec/hdfs-scan-node

[Impala-ASF-CR] IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables

2017-02-02 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5815/1/fe/src/mai

[Impala-ASF-CR] CDH-48291: Fix flaky test TestRequestPoolService

2017-02-02 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has abandoned this change. Change subject: CDH-48291: Fix flaky test TestRequestPoolService .. Abandoned MJ posted a fix for this. -- To view, visit http://gerrit.cloudera.org:8080/5507 To unsubscribe, v

[Impala-ASF-CR] IMPALA-4868: Fix flaky TestRequestPoolService.testUpdatingConfigs

2017-02-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4868: Fix flaky TestRequestPoolService.testUpdatingConfigs .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5876 To unsub

[Impala-ASF-CR] CDH-48291: Fix flaky test TestRequestPoolService

2017-02-02 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: CDH-48291: Fix flaky test TestRequestPoolService .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5507/1/fe/src/test/java/org/apache/impala/util

[Impala-ASF-CR] IMPALA-4868: Fix flaky TestRequestPoolService.testUpdatingConfigs

2017-02-02 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/5876 Change subject: IMPALA-4868: Fix flaky TestRequestPoolService.testUpdatingConfigs .. IMPALA-4868: Fix flaky TestRequestPoolService.test

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. IMPALA-4735: Upgrade pytest in python env to version 2.9.2. The current version of pytest in

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5640 To unsubscribe, vis

[Impala-ASF-CR] IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables

2017-02-02 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables .. Patch Set 2: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/5815/1/fe/src/m

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. Patch Set 9: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5640 To unsubsc

[Impala-ASF-CR] IMPALA-1427: Improvements to "Unknown disk-ID" warning

2017-02-02 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-1427: Improvements to "Unknown disk-ID" warning .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5828/3/be/src/exec/hdfs-scan-node-base.cc Fil

[Impala-ASF-CR] IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

2017-02-02 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4792: Fix number of distinct values for a CASE with constant outputs .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5768 To

[Impala-ASF-CR] IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4792: Fix number of distinct values for a CASE with constant outputs .. Patch Set 8: Build started: http://jenkins.impala.io:8080/job/gerrit-veri

[Impala-ASF-CR] IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

2017-02-02 Thread Joe McDonnell (Code Review)
Hello Marcel Kornacker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5768 to look at the new patch set (#8). Change subject: IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

[Impala-ASF-CR] IMPALA-1430: enable codegen for native UDAs

2017-02-02 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-1430: enable codegen for native UDAs .. Patch Set 13: (5 comments) http://gerrit.cloudera.org:8080/#/c/5161/13/be/src/exprs/agg-fn-evaluator.cc File be/src/e

[Impala-ASF-CR] IMPALA-1427: Improvements to "Unknown disk-ID" warning

2017-02-02 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-1427: Improvements to "Unknown disk-ID" warning .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5828/3/be/src/exec/hdfs-scan-node-

[Impala-ASF-CR] IMPALA-3406: [DOCS] Remove stale and Cloudera-specific URLs

2017-02-02 Thread Ambreen Kazi (Code Review)
Ambreen Kazi has posted comments on this change. Change subject: IMPALA-3406: [DOCS] Remove stale and Cloudera-specific URLs .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5863/1/docs/topics/impala_faq.xml Fil

[Impala-ASF-CR] IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC

2017-02-02 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5720/7/be/src/rpc/rpc

[Impala-ASF-CR] IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC

2017-02-02 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5720/4/be/src/statest

[Impala-ASF-CR] IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC

2017-02-02 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC .. Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/5720/7/be/src/rpc/

[Impala-ASF-CR] IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC

2017-02-02 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC .. Patch Set 7: This patch is a rebase. The only outstanding issue is the performance

[Impala-ASF-CR] IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC

2017-02-02 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new patch set (#7). Change subject: IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC .. IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC This patch a

[Impala-ASF-CR] IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC

2017-02-02 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC .. Patch Set 5: (3 comments) The next patch after this will be a rebase, but will con

[Impala-ASF-CR] IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC

2017-02-02 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new patch set (#6). Change subject: IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC .. IMPALA-{4670,4672,4784}: Add RpcMgr and port Statestore services to KRPC This patch a

[Impala-ASF-CR] IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

2017-02-02 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4792: Fix number of distinct values for a CASE with constant outputs .. Patch Set 7: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/57

[Impala-ASF-CR] IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables

2017-02-02 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded a new patch set (#2). Change subject: IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables .. IMPALA-2518: DROP DATABASE CASCADE removes cache directives of tables This commit

[Impala-ASF-CR] IMPALA-2518: DROP DATABASE CASCADE doesn't remove cache directives of tables

2017-02-02 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-2518: DROP DATABASE CASCADE doesn't remove cache directives of tables .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/5815/1//C

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-02-02 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/5776/8/be/src/exprs/string-functions-ir.cc File be/src/exprs/stri

[Impala-ASF-CR] IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

2017-02-02 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. Change subject: IMPALA-4792: Fix number of distinct values for a CASE with constant outputs .. Patch Set 6: (6 comments) http://gerrit.cloudera.org:8080/#/c/5768/6/fe/src/main

[Impala-ASF-CR] IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

2017-02-02 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded a new patch set (#7). Change subject: IMPALA-4792: Fix number of distinct values for a CASE with constant outputs .. IMPALA-4792: Fix number of distinct values for a CASE with constant outputs If all

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-02-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. Patch Set 9: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/228/ -

[Impala-ASF-CR] Patch references to Cloudera and CDH in Impala tutorial

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Patch references to Cloudera and CDH in Impala tutorial .. Patch Set 2: Laurel, if this addresses your -1, I am ready to +2. -- To view, visit http://gerrit.cloude

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-02-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. Patch Set 8: David, do you want me to rebase and commit this? -- To view, visit http://gerrit.cloude