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

2017-10-25 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. ( http://gerrit.cloudera.org:8080/8235 ) Change subject: IMPALA-5429: Multi threaded block metadata loading .. Patch Set 10: Code-Review+2 Nice work! -- To view,

[Impala-ASF-CR] IMPALA-3998: deprecate --refresh after connect

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8381 ) Change subject: IMPALA-3998: deprecate --refresh_after_connect .. Patch Set 1: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-3998: deprecate --refresh after connect

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8381 ) Change subject: IMPALA-3998: deprecate --refresh_after_connect .. IMPALA-3998: deprecate --refresh_after_connect This

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

2017-10-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 2: (5 comments) The code change looks good, just needs

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/7564 ) Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. Patch Set 12: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-4456: Address scalability issues of qs map lock and client request state map lock

2017-10-25 Thread Sailesh Mukil (Code Review)
Hello Philip Zeyliger, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8363 to look at the new patch set (#4). Change subject: IMPALA-4456: Address scalability issues of qs_map_lock_ and client_request_state_map_lock_

[Impala-ASF-CR] IMPALA-4456: Address scalability issues of qs map lock and client request state map lock

2017-10-25 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8363 ) Change subject: IMPALA-4456: Address scalability issues of qs_map_lock_ and client_request_state_map_lock_ .. Patch Set 4: (4

[Impala-ASF-CR] IMPALA-4856: Port data stream service to KRPC

2017-10-25 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/8023 ) Change subject: IMPALA-4856: Port data stream service to KRPC .. IMPALA-4856: Port data stream service to KRPC This patch implements a new

[Impala-ASF-CR] (PREVIEW) IMPALA-5684: Optionally run be tests in sharded mode

2017-10-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has abandoned this change. ( http://gerrit.cloudera.org:8080/7467 ) Change subject: (PREVIEW) IMPALA-5684: Optionally run be tests in sharded mode .. Abandoned Will abandon for now. I linked the CR from the JIRA

[Impala-ASF-CR] IMPALA-5315: Cast to timestamp fails for YYYY-M-D format

2017-10-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/7009 ) Change subject: IMPALA-5315: Cast to timestamp fails for -M-D format .. Patch Set 5: Do you think you'll have time to get back to

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

2017-10-25 Thread Attila Jeges (Code Review)
Attila Jeges has posted comments on this change. ( http://gerrit.cloudera.org:8080/7564 ) Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. Patch Set 11: (10 comments) Thanks for the

[Impala-ASF-CR] IMPALA-6106: handle comments before set in test parser

2017-10-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8384 Change subject: IMPALA-6106: handle comments before set in test parser .. IMPALA-6106: handle comments before set in test parser The

[Impala-ASF-CR] IMPALA-6106: handle comments before set in test parser

2017-10-25 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8384 ) Change subject: IMPALA-6106: handle comments before set in test parser .. Patch Set 1: Code-Review+2 -- To view, visit

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

2017-10-25 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 9: (16 comments)

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

2017-10-25 Thread Bharath Vissapragada (Code Review)
Hello Jim Apple, Dimitris Tsirogiannis, Mostafa Mokhtar, Alex Behm, 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 (#10). Change subject: IMPALA-5429: Multi threaded block metadata loading

[Impala-ASF-CR] IMPALA-6108: Revert "IMPALA-6070: Parallel data load."

2017-10-25 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8386 Change subject: IMPALA-6108: Revert "IMPALA-6070: Parallel data load." .. IMPALA-6108: Revert "IMPALA-6070: Parallel data load." We

[Impala-ASF-CR] IMPALA-6108: Revert "IMPALA-6070: Parallel data load."

2017-10-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8386 ) Change subject: IMPALA-6108: Revert "IMPALA-6070: Parallel data load." .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6108: Revert "IMPALA-6070: Parallel data load."

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8386 ) Change subject: IMPALA-6108: Revert "IMPALA-6070: Parallel data load." .. Patch Set 1: Build started:

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

2017-10-25 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 6: Code-Review+1 (3 comments)

[Impala-ASF-CR] IMPALA-4856: Port data stream service to KRPC

2017-10-25 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/8023 ) Change subject: IMPALA-4856: Port data stream service to KRPC .. Patch Set 4: (86 comments) Reply to some of the comments for now. Will

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

2017-10-25 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8385 Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. IMPALA-2758: Remove BufferedTupleStream::GetRows This patch removes

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

2017-10-25 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 2: Reimplemented according to the above discussion. --

[Impala-ASF-CR] IMPALA-1422: support a constant on LHS of IN predicates.

2017-10-25 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/8322 ) Change subject: IMPALA-1422: support a constant on LHS of IN predicates. .. Patch Set 5: (8 comments) missed the planner tests last

[Impala-ASF-CR] IMPALA-1422: support a constant on LHS of IN predicates.

2017-10-25 Thread Vuk Ercegovac (Code Review)
Hello Dimitris Tsirogiannis, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8322 to look at the new patch set (#6). Change subject: IMPALA-1422: support a constant on LHS of IN predicates.

[Impala-ASF-CR] IMPALA-6106: handle comments before set in test parser

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8384 ) Change subject: IMPALA-6106: handle comments before set in test parser .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

2017-10-25 Thread Tianyi Wang (Code Review)
Tianyi Wang has abandoned this change. ( http://gerrit.cloudera.org:8080/8385 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Abandoned Sorry this should be in https://gerrit.cloudera.org/c/8226/ -- To

[Impala-ASF-CR] IMPALA-2758: Change BufferedTupleStream::GetRows to returning multi batches

2017-10-25 Thread Tianyi Wang (Code Review)
Tianyi Wang has restored this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Change BufferedTupleStream::GetRows to returning multi batches .. Restored -- To view, visit

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

2017-10-25 Thread Tianyi Wang (Code Review)
Hello Thomas Tauber-Marshall, Tim Armstrong, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8226 to look at the new patch set (#2). Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

2017-10-25 Thread Tianyi Wang (Code Review)
Hello Thomas Tauber-Marshall, Tim Armstrong, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8226 to look at the new patch set (#3). Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

2017-10-25 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 3: (5 comments)

[Impala-ASF-CR] IMPALA-5019: Decimal V2 addition

2017-10-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8309 ) Change subject: IMPALA-5019: Decimal V2 addition .. Patch Set 2: (12 comments) I spent some time getting to understand the code.

[Impala-ASF-CR] IMPALA-6004: Fix test row filters failure on ASAN

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8358 ) Change subject: IMPALA-6004: Fix test_row_filters failure on ASAN .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-5736: Add impala-shell argument to set default query options

2017-10-25 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/8038 ) Change subject: IMPALA-5736: Add impala-shell argument to set default query options .. Patch Set 14: (2 comments)

[Impala-ASF-CR] IMPALA-5736: Add impala-shell argument to set default query options

2017-10-25 Thread Csaba Ringhofer (Code Review)
Hello Lars Volker, Michael Brown, Matthew Jacobs, Philip Zeyliger, anujphadke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8038 to look at the new patch set (#15). Change subject: IMPALA-5736: Add impala-shell argument to set default query options

[Impala-ASF-CR] Remove Parquet "incubating" modifier in README

2017-10-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/8378 ) Change subject: Remove Parquet "incubating" modifier in README .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] Remove Parquet "incubating" modifier in README

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8378 ) Change subject: Remove Parquet "incubating" modifier in README .. Patch Set 3: Build started:

[Impala-ASF-CR] Remove Parquet "incubating" modifier in README

2017-10-25 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/8378 ) Change subject: Remove Parquet "incubating" modifier in README .. Remove Parquet "incubating" modifier in README Change-Id:

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-25 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. Patch Set 6: Code-Review-1 The test failures are real; I'm

[Impala-ASF-CR] fix doc about parquet under incubation

2017-10-25 Thread Jim Apple (Code Review)
Jim Apple has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8378 Change subject: fix doc about parquet under incubation .. fix doc about parquet under incubation Change-Id:

[Impala-ASF-CR] IMPALA-6106: handle comments before set in test parser

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8384 ) Change subject: IMPALA-6106: handle comments before set in test parser .. IMPALA-6106: handle comments before set in

[Impala-ASF-CR] IMPALA-3436: Return a decimal when rounding a double

2017-10-25 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8398 Change subject: IMPALA-3436: Return a decimal when rounding a double .. IMPALA-3436: Return a decimal when rounding a double Before

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

2017-10-25 Thread Bharath Vissapragada (Code Review)
Hello Jim Apple, Dimitris Tsirogiannis, Mostafa Mokhtar, Alex Behm, 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 (#9). Change subject: IMPALA-5429: Multi threaded block metadata loading

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

2017-10-25 Thread Pranay Singh (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8215 to look at the new patch set (#4). Change subject: IMPALA-5142 EventSequence displays negative elapsed time. ..

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

2017-10-25 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 9: Code-Review+1 (1 comment) Thanks Alex,

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

2017-10-25 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 2: (2 comments)

[Impala-ASF-CR] IMPALA-4856: Port data stream service to KRPC

2017-10-25 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8023 ) Change subject: IMPALA-4856: Port data stream service to KRPC .. Patch Set 3: (16 comments) Some more comments, still going though.

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

2017-10-25 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/8202 ) Change subject: IMPALA-4704: Turns on client connections when local catalog initialized. .. Patch Set 12: (11 comments) main change

[Impala-ASF-CR] first cut at comments

2017-10-25 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8379 Change subject: first cut at comments .. first cut at comments Change-Id: I3f77d2b8dffa6fa8931e62171ba5968557823c5a --- M

[Impala-ASF-CR] ok

2017-10-25 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8380 Change subject: ok .. ok Change-Id: I4ed4b7705eef3194b77442be69a903a183b51c9b --- M be/src/service/impala-server.cc M

[Impala-ASF-CR] IMPALA-3998: deprecate --refresh after connect

2017-10-25 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/8381 ) Change subject: IMPALA-3998: deprecate --refresh_after_connect .. Patch Set 1: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-1422: support a constant on LHS of IN predicates.

2017-10-25 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. ( http://gerrit.cloudera.org:8080/8322 ) Change subject: IMPALA-1422: support a constant on LHS of IN predicates. .. Patch Set 5: (8 comments) Did you add any planner

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

2017-10-25 Thread Attila Jeges (Code Review)
Attila Jeges has uploaded a new patch set (#12). ( http://gerrit.cloudera.org:8080/7564 ) Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. IMPALA-3548: Prune runtime filters based on query

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

2017-10-25 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8202 ) Change subject: IMPALA-4704: Turns on client connections when local catalog initialized. .. Patch Set 12: (1 comment)

[Impala-ASF-CR] first cut at comments

2017-10-25 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has abandoned this change. ( http://gerrit.cloudera.org:8080/8379 ) Change subject: first cut at comments .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/8379 To unsubscribe, visit

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

2017-10-25 Thread Vuk Ercegovac (Code Review)
Hello Michael Brown, Philip Zeyliger, Balazs Jeszenszky, Dimitris Tsirogiannis, Alex Behm, 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 (#13). Change subject: IMPALA-4704: Turns on client connections when

[Impala-ASF-CR] ok

2017-10-25 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has abandoned this change. ( http://gerrit.cloudera.org:8080/8380 ) Change subject: ok .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/8380 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3998: deprecate --refresh after connect

2017-10-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8381 Change subject: IMPALA-3998: deprecate --refresh_after_connect .. IMPALA-3998: deprecate --refresh_after_connect This adds a warning to

[Impala-ASF-CR] IMPALA-1291: Parquet read fails if io buffer size is less than the footer size

2017-10-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8366 ) Change subject: IMPALA-1291: Parquet read fails if io buffer size is less than the footer size .. Patch Set 2: Code-Review+2 (1

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

2017-10-25 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 2: (2 comments)

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

2017-10-25 Thread Pranay Singh (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8215 to look at the new patch set (#3). Change subject: IMPALA-5142 EventSequence displays negative elapsed time. ..

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/7564 ) Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. Patch Set 12: Build started:

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

2017-10-25 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/7564 ) Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. Patch Set 12: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-5736: Add impala-shell argument to set default query options

2017-10-25 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/8038 ) Change subject: IMPALA-5736: Add impala-shell argument to set default query options .. Patch Set 15: Code-Review+1 Thank you for the

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

2017-10-25 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/8202 ) Change subject: IMPALA-4704: Turns on client connections when local catalog initialized. .. Patch Set 13: (1 comment)

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

2017-10-25 Thread Vuk Ercegovac (Code Review)
Hello Michael Brown, Philip Zeyliger, Balazs Jeszenszky, Dimitris Tsirogiannis, Alex Behm, 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 (#14). Change subject: IMPALA-4704: Turns on client connections when

[Impala-ASF-CR] IMPALA-3998: deprecate --refresh after connect

2017-10-25 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/8381 ) Change subject: IMPALA-3998: deprecate --refresh_after_connect .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-5736: Add impala-shell argument to set default query options

2017-10-25 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8038 ) Change subject: IMPALA-5736: Add impala-shell argument to set default query options .. Patch Set 15: (6 comments) I'm lukewarm on

[Impala-ASF-CR] DRAFT IMPALA-5185: Skip pages based on Parquet::Statistics

2017-10-25 Thread Lars Volker (Code Review)
Lars Volker has abandoned this change. ( http://gerrit.cloudera.org:8080/7354 ) Change subject: DRAFT IMPALA-5185: Skip pages based on Parquet::Statistics .. Abandoned IMPALA-5185 has been closed as Won't Fix. -- To view,

[Impala-ASF-CR] IMPALA-6004: Fix test row filters failure on ASAN

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8358 ) Change subject: IMPALA-6004: Fix test_row_filters failure on ASAN .. IMPALA-6004: Fix test_row_filters failure on ASAN

[Impala-ASF-CR] IMPALA-6004: Fix test row filters failure on ASAN

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8358 ) Change subject: IMPALA-6004: Fix test_row_filters failure on ASAN .. Patch Set 2: Verified+1 -- To view, visit

[Impala-ASF-CR] Remove Parquet "incubating" modifier in README

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8378 ) Change subject: Remove Parquet "incubating" modifier in README .. Patch Set 3: Verified+1 -- To view, visit

[Impala-ASF-CR] Remove Parquet "incubating" modifier in README

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8378 ) Change subject: Remove Parquet "incubating" modifier in README .. Remove Parquet "incubating" modifier in README

[Impala-ASF-CR] DRAFT IMPALA-5185: Skip pages based on Parquet::Statistics

2017-10-25 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/7354 ) Change subject: DRAFT IMPALA-5185: Skip pages based on Parquet::Statistics .. Patch Set 3: Yes, thanks for the reminder. I'll push the

[Impala-ASF-CR] IMPALA-3998: deprecate --refresh after connect

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8381 ) Change subject: IMPALA-3998: deprecate --refresh_after_connect .. Patch Set 1: Build started:

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

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

[Impala-ASF-CR] IMPALA-6106: handle comments before set in test parser

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8384 ) Change subject: IMPALA-6106: handle comments before set in test parser .. Patch Set 1: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6108: Revert "IMPALA-6070: Parallel data load."

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8386 ) Change subject: IMPALA-6108: Revert "IMPALA-6070: Parallel data load." .. IMPALA-6108: Revert "IMPALA-6070: Parallel

[Impala-ASF-CR] IMPALA-6108: Revert "IMPALA-6070: Parallel data load."

2017-10-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8386 ) Change subject: IMPALA-6108: Revert "IMPALA-6070: Parallel data load." .. Patch Set 1: Verified+1 -- To view, visit

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

2017-10-25 Thread Sailesh Mukil (Code Review)
Hello Michael Ho, Bikramjeet Vig, Dan Hecht, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7938 to look at the new patch set (#12). Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork

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

2017-10-25 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/7938 ) Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork .. Patch Set 12: The GVO was hanging since the kerberos

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

2017-10-25 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/7938 ) Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork .. Patch Set 12: (1 comment)

[Impala-ASF-CR] IMPALA-1291: Parquet read fails if io buffer size is less than the footer size

2017-10-25 Thread Zoltan Borok-Nagy (Code Review)
Zoltan Borok-Nagy has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8366 Change subject: IMPALA-1291: Parquet read fails if io buffer size is less than the footer size .. IMPALA-1291: Parquet read fails