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

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool .. Patch Set 15: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/384/

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

2017-03-15 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5811 to look at the new patch set (#15). Change subject: IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool

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

2017-03-15 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 15: Code-Review+2 Fix some test failures from the rebase -- To view, visit http://gerr

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. IMPALA-4787: Optimize APPX_MEDIAN() memory usage Before this change, ReservoirSample functio

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 17: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6025 To unsubscribe, visi

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 8: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/383/ -- To view,

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 8: Code-Review+2 Fix a linker issue - order of boost libraries was wrong -- To view, visit http:/

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Jim Apple, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6402 to look at the new patch set (#8). Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .

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

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool .. Patch Set 14: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dry

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 7: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/380/ --

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

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool .. Patch Set 14: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/382/

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

2017-03-15 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 14: Code-Review+2 Carry +2 -- To view, visit http://gerrit.cloudera.org:8080/5811 To u

[Impala-ASF-CR] IMPALA-4996: Single-threaded KuduScanNode

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4996: Single-threaded KuduScanNode .. Patch Set 6: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/377/ -- To vi

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 17: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/381/ -- To view

[Impala-ASF-CR] IMPALA-4831: enforce BufferPool reservation invariants

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4831: enforce BufferPool reservation invariants .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6313 To unsubscribe,

[Impala-ASF-CR] IMPALA-4831: enforce BufferPool reservation invariants

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4831: enforce BufferPool reservation invariants .. IMPALA-4831: enforce BufferPool reservation invariants Before this patch ill-behaved c

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-03-15 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 17: Code-Review+2 Forgot to update the alloc test. Made the tiny change to the test. Forwarding

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-03-15 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#17). Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. IMPALA-4787: Optimize APPX_MEDIAN() memory usage Before this change, ReservoirSample functions (such as AP

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-03-15 Thread Taras Bobrovytsky (Code Review)
Hello Marcel Kornacker, Impala Public Jenkins, Matthew Jacobs, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6025 to look at the new patch set (#17). Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .

[native-toolchain-CR] IMPALA-5057: Upgrade gflags (2.2.0) and glog (0.3.4-p2)

2017-03-15 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5057: Upgrade gflags (2.2.0) and glog (0.3.4-p2) .. Patch Set 2: Lars took a look at the glog breakpad patch, and Matt took a quick look at the glog mes

[native-toolchain-CR] IMPALA-5057: Upgrade gflags (2.2.0) and glog (0.3.4-p2)

2017-03-15 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5057: Upgrade gflags (2.2.0) and glog (0.3.4-p2) .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6372/2/source/gflags/build.sh File sour

[native-toolchain-CR] IMPALA-5057: Upgrade gflags (2.2.0) and glog (0.3.4-p2)

2017-03-15 Thread Henry Robinson (Code Review)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6372 to look at the new patch set (#3). Change subject: IMPALA-5057: Upgrade gflags (2.2.0) and glog (0.3.4-p2) .. IMP

[Impala-ASF-CR] IMPALA-3203: Part 1: Free list implementation

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/6410 Change subject: IMPALA-3203: Part 1: Free list implementation .. IMPALA-3203: Part 1: Free list implementation We will have a single fre

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 16: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/378/ -

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

2017-03-15 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 13: Code-Review+2 Rebased onto https://gerrit.cloudera.org/#/c/6313/ -- To view, visit

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

2017-03-15 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5811 to look at the new patch set (#13). Change subject: IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool .. IMP

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

2017-03-15 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 12: Code-Review+2 Carry +2 -- To view, visit http://gerrit.cloudera.org:8080/5811 To u

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 7: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/380/ -- To view,

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

2017-03-15 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 11: (7 comments) http://gerrit.cloudera.org:8080/#/c/5811/11/be/src/runtime/buffered-tu

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

2017-03-15 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5811 to look at the new patch set (#12). Change subject: IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool .. IMP

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6402 To unsubscribe, visit http://g

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 7: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6402 To unsubscribe, visit http://g

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-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/6402 to look at the new patch set (#7). Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. IMPA

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/6402/6/be/src/util/cpu-info.cc File be/src/util

[Impala-ASF-CR] IMPALA-4029: Reduce memory requirements for storing file metadata

2017-03-15 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4029: Reduce memory requirements for storing file metadata .. Patch Set 1: (14 comments) http://gerrit.cloudera.org:8080/#/c/6406/1//COMMIT_MSG C

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

2017-03-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool .. Patch Set 11: Code-Review+2 (7 comments) http://gerrit.cloudera.org:8080/#/c/5811/11/be/src/runtime/b

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/6402/6/be/src/util/cpu-info.cc File be/src/util/cpu

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-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/6402 to look at the new patch set (#6). Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. IMPA

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/6402/5/be/src/util/cpu-info.cc File be/src/uti

[Impala-ASF-CR] [DOCS] Pare down ODBC info

2017-03-15 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: [DOCS] Pare down ODBC info .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6409/1/docs/topics/impala_odbc.xml File docs/topics/impala_odbc.xml: PS

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

2017-03-15 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 10: (19 comments) http://gerrit.cloudera.org:8080/#/c/5811/9/be/src/runtime/buffered-tu

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

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

[Impala-ASF-CR] [DOCS] Pare down ODBC info

2017-03-15 Thread Greg Rahn (Code Review)
Greg Rahn has posted comments on this change. Change subject: [DOCS] Pare down ODBC info .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6409 To unsubscribe, visit http://gerrit.cloudera.org:808

[Impala-ASF-CR] [DOCS] Pare down ODBC info

2017-03-15 Thread John Russell (Code Review)
John Russell has uploaded a new change for review. http://gerrit.cloudera.org:8080/6409 Change subject: [DOCS] Pare down ODBC info .. [DOCS] Pare down ODBC info Remove the Cloudera-specific portion of the Impala + ODBC page,

[Impala-ASF-CR] IMPALA-4831: enforce BufferPool reservation invariants

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4831: enforce BufferPool reservation invariants .. Patch Set 3: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/379/ -- To

[Impala-ASF-CR] IMPALA-4318: Kudu support for CREATE EXTERNAL TABLE AS SELECT

2017-03-15 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4318: Kudu support for CREATE EXTERNAL TABLE AS SELECT .. Patch Set 6: (3 comments) http://gerrit.cloudera.org:8080/#/c/6261/6/fe/src/main/java/org/apache

[Impala-ASF-CR] IMPALA-5072: Fix test recover partitions on S3

2017-03-15 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-5072: Fix test_recover_partitions on S3 .. Patch Set 1: Code-Review+2 In terms of testing, you may run a private jenkins jobs to test your uncomm

[Impala-ASF-CR] IMPALA-5072: Fix test recover partitions on S3

2017-03-15 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/6408 Change subject: IMPALA-5072: Fix test_recover_partitions on S3 .. IMPALA-5072: Fix test_recover_partitions on S3 On S3, directory creation

[Impala-ASF-CR] IMPALA-4831: enforce BufferPool reservation invariants

2017-03-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4831: enforce BufferPool reservation invariants .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6313 To unsubscribe, visit ht

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 5: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/6402/5/be/src/util/cpu-info.cc File

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-03-15 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 16: Thanks for all the reviews, Alex, Jim, MJ, Mostafa, Marcel! -- To view, visit http://gerr

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 16: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/378/ -- To view

[Impala-ASF-CR] IMPALA-3586: Implement union passthrough

2017-03-15 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3586: Implement union passthrough .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/5816/9//COMMIT_MSG Commit Message: Line 15: as a precautio

[Impala-ASF-CR] IMPALA-3748: minimum buffer requirements in planner

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3748: minimum buffer requirements in planner .. Patch Set 13: Updated the explain plan output to use more concise names for estimates and reservations. R

[Impala-ASF-CR] IMPALA-3748: minimum buffer requirements in planner

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#13). Change subject: IMPALA-3748: minimum buffer requirements in planner .. IMPALA-3748: minimum buffer requirements in planner Compute the minimum buffer requirement for spilling nod

[Impala-ASF-CR] IMPALA-3586: Implement union passthrough

2017-03-15 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-3586: Implement union passthrough .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/5816/9//COMMIT_MSG Commit Message: Line 15: as a pr

[Impala-ASF-CR] IMPALA-3586: Implement union passthrough

2017-03-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3586: Implement union passthrough .. Patch Set 16: (1 comment) http://gerrit.cloudera.org:8080/#/c/5816/16/be/src/exec/union-node.h File be/src/exec/union-no

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

2017-03-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4674: Part 1: port BufferedTupleStream to BufferPool .. Patch Set 10: (19 comments) http://gerrit.cloudera.org:8080/#/c/5811/9/be/src/runtime/buffered-tuple-

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. Patch Set 8: (4 comments) http://gerrit.cloudera.org:8080/#/c/6037/7/be/src/runtime/data-stream-partiti

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#8). Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. IMPALA-3742: partitions DMLs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE, and DELETE) for Kudu are c

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 5: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6402 To unsubscribe, visit http

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Tim Armstrong (Code Review)
Hello Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6402 to look at the new patch set (#5). Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. IMPALA-5077: ad

[Impala-ASF-CR] IMPALA-4029: Reduce memory requirements for storing file metadata

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4029: Reduce memory requirements for storing file metadata .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6406/1/fe/src/main/java/org/a

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 3: Code-Review+1 Carry Jim's +1 -- To view, visit http://gerrit.cloudera.org:8080/6402 To unsubsc

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/6402/3/be/CMakeLists.txt File be/CMakeLists.tx

[Impala-ASF-CR] IMPALA-4029: Reduce memory requirements for storing file metadata

2017-03-15 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded a new change for review. http://gerrit.cloudera.org:8080/6406 Change subject: IMPALA-4029: Reduce memory requirements for storing file metadata .. IMPALA-4029: Reduce memory requirements for

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/6402/3/be/CMakeLists.txt File be/CMakeLists.t

[Impala-ASF-CR] IMPALA-4996: Single-threaded KuduScanNode

2017-03-15 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4996: Single-threaded KuduScanNode .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6312 To unsubscribe, visit http://g

[Impala-ASF-CR] IMPALA-4996: Single-threaded KuduScanNode

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4996: Single-threaded KuduScanNode .. Patch Set 6: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/377/ -- To view, visit

[Impala-ASF-CR] IMPALA-4996: Single-threaded KuduScanNode

2017-03-15 Thread Joe McDonnell (Code Review)
Hello Marcel Kornacker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6312 to look at the new patch set (#6). Change subject: IMPALA-4996: Single-threaded KuduScanNode .. IMPALA-4996: Si

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-03-15 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 16: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6025 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6402 To unsubscribe, visit http://g

[Impala-ASF-CR] IMPALA-4996: Single-threaded KuduScanNode

2017-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4996: Single-threaded KuduScanNode .. Patch Set 5: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/376/ -- To view, visit

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-03-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs Previously, exprs used in sorts w

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-03-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. Patch Set 2: (24 comments) http://gerrit.cloudera.org:8080/#/c/6322/1/fe/src/main/java/or

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#3). Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. IMPALA-5077: add NUMA and current cpu to CpuInfo NUMA information is found using the /sys filesystem. The curr

[Impala-ASF-CR] IMPALA-5077: add NUMA and current cpu to CpuInfo

2017-03-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5077: add NUMA and current cpu to CpuInfo .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/6402/2/be/src/util/cpu-info.cc File be/src/uti