[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

[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

[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:

[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

[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

[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

[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

[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

[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.

[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

[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

[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

[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) ..

[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

[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,

[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 ..

[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

[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

[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)

[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 ..

[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

[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

[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 ..

[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

[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

[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)

[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

[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 ..

[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

[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:

[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)

[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

[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/ --

[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)

[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

[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

[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

[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

[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

[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

[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.

[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

[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

[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

[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)

[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)

[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

[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

[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:

[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)

[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

[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

[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

[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

[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

[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:

[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

[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

[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)

[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

[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