[Impala-ASF-CR] IMPALA-6045: Make build scripts more friendly to Ubuntu 16.04

2017-10-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/8262 ) Change subject: IMPALA-6045: Make build scripts more friendly to Ubuntu 16.04 .. Patch Set 3: > Sorry to get to this a bit late. No

[Impala-ASF-CR] Making bin/bootstrap system.sh executable.

2017-10-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/8292 ) Change subject: Making bin/bootstrap_system.sh executable. .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] Reduce log spew from rpcz store.cc

2017-10-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8285 ) Change subject: Reduce log spew from rpcz_store.cc .. Patch Set 1: Verified+1 -- To view, visit

[Impala-ASF-CR] Reduce log spew from rpcz store.cc

2017-10-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8285 ) Change subject: Reduce log spew from rpcz_store.cc .. Reduce log spew from rpcz_store.cc The sampled RPC call

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-10-16 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8215 ) Change subject: IMPALA-5142 EventSequence displays negative elapsed time. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] Making bin/bootstrap system.sh executable.

2017-10-16 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8292 Change subject: Making bin/bootstrap_system.sh executable. .. Making bin/bootstrap_system.sh executable. Script should be executable,

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-16 Thread Bikramjeet Vig (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8196 to look at the new patch set (#8). Change subject: IMPALA-4236: Codegen CopyRows() for select nodes ..

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-16 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 7: (3 comments)

[Impala-ASF-CR] IMPALA-4524: Batch calls to ALTER TABLE...ADD PARTITION.

2017-10-16 Thread Philip Zeyliger (Code Review)
Hello Dimitris Tsirogiannis, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8238 to look at the new patch set (#5). Change subject: IMPALA-4524: Batch calls to ALTER TABLE...ADD PARTITION.

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-10-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8215 ) Change subject: IMPALA-5142 EventSequence displays negative elapsed time. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6002: Add a LLVM diagnostic handler for LLVM linker errors

2017-10-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8233 ) Change subject: IMPALA-6002: Add a LLVM diagnostic handler for LLVM linker errors .. Patch Set 4: Code-Review+2 (1 comment)

[Impala-ASF-CR] Reduce log spew from rpcz store.cc

2017-10-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8285 ) Change subject: Reduce log spew from rpcz_store.cc .. Patch Set 1: Build started:

[Impala-ASF-CR] Reduce log spew from rpcz store.cc

2017-10-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8285 ) Change subject: Reduce log spew from rpcz_store.cc .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8285

[Impala-ASF-CR] IMPALA-5129: Use KRPC's Kinit code to avoid expensive fork

2017-10-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/7938 ) Change subject: IMPALA-5129: Use KRPC's Kinit code to avoid expensive fork .. Patch Set 8: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-5429: Multi threaded block metadata loading

2017-10-16 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8235 ) Change subject: IMPALA-5429: Multi threaded block metadata loading .. Patch Set 6: (15 comments)

[Impala-ASF-CR] IMPALA-6002: Add a LLVM diagnostic handler for LLVM linker errors

2017-10-16 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/8233 ) Change subject: IMPALA-6002: Add a LLVM diagnostic handler for LLVM linker errors .. Patch Set 4: Code-Review+1 (2 comments)

[Impala-ASF-CR] IMPALA-4524: Batch calls to ALTER TABLE...ADD PARTITION.

2017-10-16 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. ( http://gerrit.cloudera.org:8080/8238 ) Change subject: IMPALA-4524: Batch calls to ALTER TABLE...ADD PARTITION. .. Patch Set 4: Code-Review+2 (2 comments)

[Impala-ASF-CR] IMPALA-6002: Add a LLVM diagnostic handler for LLVM linker errors

2017-10-16 Thread Bikramjeet Vig (Code Review)
Hello Philip Zeyliger, anujphadke, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8233 to look at the new patch set (#4). Change subject: IMPALA-6002: Add a LLVM diagnostic handler for LLVM linker errors

[Impala-ASF-CR] IMPALA-6002: Add a LLVM diagnostic handler for LLVM linker errors

2017-10-16 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8233 ) Change subject: IMPALA-6002: Add a LLVM diagnostic handler for LLVM linker errors .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-6002: Add a LLVM diagnostic handler for LLVM linker errors

2017-10-16 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8233 ) Change subject: IMPALA-6002: Add a LLVM diagnostic handler for LLVM linker errors .. Patch Set 3: (5 comments)

[Impala-ASF-CR] IMPALA-5429: Multi threaded block metadata loading

2017-10-16 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/8235 ) Change subject: IMPALA-5429: Multi threaded block metadata loading .. Patch Set 6: (4 comments)

[Impala-ASF-CR] Reduce log spew from rpcz store.cc

2017-10-16 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8285 ) Change subject: Reduce log spew from rpcz_store.cc .. Patch Set 1: > Uploaded patch set 1. This cherry-pick was clean and is intended

[Impala-ASF-CR] Reduce log spew from rpcz store.cc

2017-10-16 Thread Sailesh Mukil (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8285 to review the following change. Change subject: Reduce log spew from rpcz_store.cc .. Reduce log spew

[Impala-ASF-CR] IMPALA-2494: Support for byte array encoded decimals in Parquet scanner

2017-10-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/7822 ) Change subject: IMPALA-2494: Support for byte array encoded decimals in Parquet scanner .. Patch Set 4: (3 comments) Few more

[Impala-ASF-CR] IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+

2017-10-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8274 ) Change subject: IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+ .. Patch Set 2: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-5429: Multi threaded block metadata loading

2017-10-16 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/8235 ) Change subject: IMPALA-5429: Multi threaded block metadata loading .. Patch Set 6: (4 comments)

[Impala-ASF-CR] IMPALA-4524: Batch calls to ALTER TABLE...ADD PARTITION.

2017-10-16 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/8238 ) Change subject: IMPALA-4524: Batch calls to ALTER TABLE...ADD PARTITION. .. Patch Set 4: (2 comments) Thanks for the review! I ran

[Impala-ASF-CR] IMPALA-4524: Batch calls to ALTER TABLE...ADD PARTITION.

2017-10-16 Thread Philip Zeyliger (Code Review)
Hello Dimitris Tsirogiannis, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8238 to look at the new patch set (#4). Change subject: IMPALA-4524: Batch calls to ALTER TABLE...ADD PARTITION.

[Impala-ASF-CR] IMPALA-5307: Part 4: copy out uncompressed text and seq

2017-10-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/8172 ) Change subject: IMPALA-5307: Part 4: copy out uncompressed text and seq .. IMPALA-5307: Part 4: copy out uncompressed text and seq This

[Impala-ASF-CR] IMPALA-5429: Multi threaded block metadata loading

2017-10-16 Thread Bharath Vissapragada (Code Review)
Hello Jim Apple, Dimitris Tsirogiannis, Mostafa Mokhtar, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8235 to look at the new patch set (#6). Change subject: IMPALA-5429: Multi threaded block metadata loading

[Impala-ASF-CR] IMPALA-5429: Multi threaded block metadata loading

2017-10-16 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/8235 ) Change subject: IMPALA-5429: Multi threaded block metadata loading .. Patch Set 5: (36 comments)

[Impala-ASF-CR] IMPALA-4704: Turns on client connections when local catalog initialized.

2017-10-16 Thread Vuk Ercegovac (Code Review)
Hello Philip Zeyliger, Balazs Jeszenszky, Dimitris Tsirogiannis, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8202 to look at the new patch set (#8). Change subject: IMPALA-4704: Turns on client connections when local catalog initialized.

[Impala-ASF-CR] IMPALA-4704: Turns on client connections when local catalog initialized.

2017-10-16 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has abandoned this change. ( http://gerrit.cloudera.org:8080/8275 ) Change subject: IMPALA-4704: Turns on client connections when local catalog initialized. .. Abandoned -- To view, visit

[Impala-ASF-CR] IMPALA-4704: Turns on client connections when local catalog initialized.

2017-10-16 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/8275 ) Change subject: IMPALA-4704: Turns on client connections when local catalog initialized. .. IMPALA-4704: Turns on client connections

[Impala-ASF-CR] IMPALA-4704: Disallow client connections until local catalog initialized.

2017-10-16 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/8275 ) Change subject: IMPALA-4704: Disallow client connections until local catalog initialized. .. IMPALA-4704: Disallow client connections

[Impala-ASF-CR] IMPALA-4704: Disallow client connections until local catalog initialized.

2017-10-16 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/8275 ) Change subject: IMPALA-4704: Disallow client connections until local catalog initialized. .. IMPALA-4704: Disallow client connections

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-10-16 Thread Anonymous Coward (Code Review)
Anonymous Coward #345 has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 6: (1 comment) link error while not include this