[Impala-ASF-CR] IMPALA-5229: huge page-backed buffers with TCMalloc

2017-04-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5229: huge page-backed buffers with TCMalloc .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6687 To unsubscribe,

[Impala-ASF-CR] IMPALA-5192: Don't bake MemPool* into IR

2017-04-21 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5192: Don't bake MemPool* into IR .. Patch Set 4: Code-Review+1 Carry +1 forward. -- To view, visit http://gerrit.cloudera.org:8080/6657 To unsubscribe,

[Impala-ASF-CR] IMPALA-5229: huge page-backed buffers with TCMalloc

2017-04-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5229: huge page-backed buffers with TCMalloc .. IMPALA-5229: huge page-backed buffers with TCMalloc This commit enables transparent huge

[Impala-ASF-CR] IMPALA-5222: don't call Bits::Log2*() functions

2017-04-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5222: don't call Bits::Log2*() functions .. IMPALA-5222: don't call Bits::Log2*() functions Some of gutil's functions are fairly

[Impala-ASF-CR] IMPALA-5222: don't call Bits::Log2*() functions

2017-04-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5222: don't call Bits::Log2*() functions .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6675 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4943: Speed up block md loading for add/recover partition calls.

2017-04-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4943: Speed up block md loading for add/recover partition calls. .. Patch Set 5: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-2550: Switch to per-query exec rpc

2017-04-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-2550: Switch to per-query exec rpc .. Patch Set 9: (23 comments) Did a review pass while I don't have access to my desktop.

[Impala-ASF-CR] IMPALA-5152: Gather all tables with missing metadata in analysis

2017-04-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5152: Gather all tables with missing metadata in analysis .. Patch Set 1: (4 comments)

[Impala-ASF-CR] IMPALA-2550: Switch to per-query exec rpc

2017-04-21 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-2550: Switch to per-query exec rpc .. Patch Set 9: (4 comments) http://gerrit.cloudera.org:8080/#/c/6535/9/be/src/runtime/coordinator.cc File

[Impala-ASF-CR] IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables.

2017-04-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables. .. Patch Set 2: > I added a test case, including a parquet file containing ENUM. I >

[Impala-ASF-CR] IMPALA-5229: huge page-backed buffers with TCMalloc

2017-04-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5229: huge page-backed buffers with TCMalloc .. Patch Set 6: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/493/ -- To

[Impala-ASF-CR] IMPALA-5229: huge page-backed buffers with TCMalloc

2017-04-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5229: huge page-backed buffers with TCMalloc .. Patch Set 5: Code-Review+2 Rebase -- To view, visit http://gerrit.cloudera.org:8080/6687 To

[Impala-ASF-CR] IMPALA-5229: huge page-backed buffers with TCMalloc

2017-04-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5229: huge page-backed buffers with TCMalloc .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6687 To unsubscribe, visit

[native-toolchain-CR] Ported native-toolchain to work on ppc64le

2017-04-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Ported native-toolchain to work on ppc64le .. Patch Set 1: (1 comment) Thanks for the responses - looking forward to the next patchset. I missed that you were

[Impala-ASF-CR] IMPALA-5222: don't call Bits::Log2*() functions

2017-04-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5222: don't call Bits::Log2*() functions .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/492/ -- To view,

[Impala-ASF-CR] IMPALA-4943: Speed up block md loading for add/recover partition calls.

2017-04-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4943: Speed up block md loading for add/recover partition calls. .. Patch Set 5: Build started:

[Impala-ASF-CR] IMPALA-4943: Speed up block md loading for add/recover partition calls.

2017-04-21 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4943: Speed up block md loading for add/recover partition calls. .. Patch Set 5: Discussed this with Alex, this looks like one of those flaky

[Impala-ASF-CR] IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables.

2017-04-21 Thread Jakub Kukul (Code Review)
Jakub Kukul has abandoned this change. Change subject: IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/6712 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables.

2017-04-21 Thread Jakub Kukul (Code Review)
Jakub Kukul has posted comments on this change. Change subject: IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables. .. Patch Set 2: I added a test case, including a parquet file containing ENUM. I haven't

[Impala-ASF-CR] IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables.

2017-04-21 Thread Jakub Kukul (Code Review)
Jakub Kukul has restored this change. Change subject: IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables. .. Restored -- To view, visit http://gerrit.cloudera.org:8080/6712 To unsubscribe, visit

[native-toolchain-CR] Bumped Kudu version to 238249c.

2017-04-21 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: Bumped Kudu version to 238249c. .. Bumped Kudu version to 238249c. This will pull in the Kudu client partitioner API, which is needed for IMPALA-3742.

[Impala-ASF-CR] IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables.

2017-04-21 Thread Jakub Kukul (Code Review)
Jakub Kukul has abandoned this change. Change subject: IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables. .. Abandoned Sorry, pushed by accident. -- To view, visit http://gerrit.cloudera.org:8080/6712

[native-toolchain-CR] Bumped Kudu version to 238249c.

2017-04-21 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Bumped Kudu version to 238249c. .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6711 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables.

2017-04-21 Thread Jakub Kukul (Code Review)
Jakub Kukul has uploaded a new patch set (#2). Change subject: IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables. .. IMPALA-2525: Treat parquet ENUMs as STRINGs when creating impala tables. Change-Id:

[native-toolchain-CR] Bumped Kudu version to 238249c.

2017-04-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bumped Kudu version to 238249c. .. Patch Set 1: This passed a toolchain verify build:

[native-toolchain-CR] Bumped Kudu version to 238249c.

2017-04-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6711 Change subject: Bumped Kudu version to 238249c. .. Bumped Kudu version to 238249c. This will pull in the Kudu client

[Impala-ASF-CR] IMPALA-5192: Don't bake MemPool* into IR

2017-04-21 Thread Michael Ho (Code Review)
Hello Taras Bobrovytsky, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6657 to look at the new patch set (#4). Change subject: IMPALA-5192: Don't bake MemPool* into IR ..