[Impala-ASF-CR] [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls

2018-06-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10696 Change subject: [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls .. [DRAFT] IMPALA-6189: Add thread watchdogs for

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-20 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 7: Code-Review+1 (7 comments) > Patch Set 5: Code-Review+1 > >

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-20 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#7). ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. IMPALA-7180: Pin Impala CDH dependencies For IMPALA_MINICLUSTER_PROFILE=3

[Impala-ASF-CR] IMPALA-5202: Disallow PREPARE:WAIT debug action

2018-06-20 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10776 ) Change subject: IMPALA-5202: Disallow PREPARE:WAIT debug action .. Patch Set 2: Code-Review+2 Rebased onto master -- To view, visit

[Impala-ASF-CR] IMPALA-5202: Disallow PREPARE:WAIT debug action

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10776 ) Change subject: IMPALA-5202: Disallow PREPARE:WAIT debug action .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-6642 (Part 2): Add timestamps to start-impala-cluster.py

2018-06-20 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10780 ) Change subject: IMPALA-6642 (Part 2): Add timestamps to start-impala-cluster.py .. Patch Set 1: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-6642 (Part 2): Add timestamps to start-impala-cluster.py

2018-06-20 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10780 ) Change subject: IMPALA-6642 (Part 2): Add timestamps to start-impala-cluster.py .. Patch Set 1: This seems fine, but using Python

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-20 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 5: Code-Review+1 (7 comments) I think this is fine. I have

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-20 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 5: (14 comments)

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-20 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. IMPALA-7180: Pin Impala CDH dependencies For IMPALA_MINICLUSTER_PROFILE=3

[Impala-ASF-CR] IMPALA-5202: Disallow PREPARE:WAIT debug action

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10776 ) Change subject: IMPALA-5202: Disallow PREPARE:WAIT debug action .. Patch Set 1: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7175: deflake check for failed impalad

2018-06-20 Thread Vuk Ercegovac (Code Review)
Hello Csaba Ringhofer, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10745 to look at the new patch set (#2). Change subject: IMPALA-7175: deflake check for failed impalad ..

[Impala-ASF-CR] IMPALA-7175: deflake check for failed impalad

2018-06-20 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10745 ) Change subject: IMPALA-7175: deflake check for failed impalad .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6642 (Part 2): Add timestamps to start-impala-cluster.py

2018-06-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10780 Change subject: IMPALA-6642 (Part 2): Add timestamps to start-impala-cluster.py .. IMPALA-6642 (Part 2): Add timestamps to

[Impala-ASF-CR] [DOCS] Added another scenario when REFRESH is required

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10773 ) Change subject: [DOCS] Added another scenario when REFRESH is required .. Patch Set 2: Verified+1 -- To view, visit

[Impala-ASF-CR] [DOCS] Added another scenario when REFRESH is required

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10773 ) Change subject: [DOCS] Added another scenario when REFRESH is required .. [DOCS] Added another scenario when REFRESH

[Impala-ASF-CR] [DOCS] Added another scenario when REFRESH is required

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10773 ) Change subject: [DOCS] Added another scenario when REFRESH is required .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-7187: Fix test group impersonation running inside Docker

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10772 ) Change subject: IMPALA-7187: Fix test_group_impersonation running inside Docker .. Patch Set 4: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7187: Fix test group impersonation running inside Docker

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10772 ) Change subject: IMPALA-7187: Fix test_group_impersonation running inside Docker .. IMPALA-7187: Fix

[Impala-ASF-CR] IMPALA-110 (part 2): Refactor PartitionedAggregationNode

2018-06-20 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10394 ) Change subject: IMPALA-110 (part 2): Refactor PartitionedAggregationNode .. Patch Set 3: (4 comments)

[Impala-ASF-CR] [DOCS] Added another scenario when REFRESH is required

2018-06-20 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10773 ) Change subject: [DOCS] Added another scenario when REFRESH is required .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6119: Fix issue with multiple partitions sharing same location

2018-06-20 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10543 ) Change subject: IMPALA-6119: Fix issue with multiple partitions sharing same location .. Patch Set 17: (9 comments)

[Impala-ASF-CR] [DOCS] Added another scenario when REFRESH is required

2018-06-20 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10773 ) Change subject: [DOCS] Added another scenario when REFRESH is required .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7046: introduce "global" debug actions

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10690 ) Change subject: IMPALA-7046: introduce "global" debug_actions .. Patch Set 13: (1 comment)

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-20 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/10778 ) Change subject: IMPALA-7191: don't call srand() at random times .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-4848: Add WIDTH BUCKET() function

2018-06-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. ( http://gerrit.cloudera.org:8080/6023 ) Change subject: IMPALA-4848: Add WIDTH_BUCKET() function .. Patch Set 15: Also, I agree with Tim that it's a good idea to add this

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-20 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10778 ) Change subject: IMPALA-7191: don't call srand() at random times .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-20 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/10778 ) Change subject: IMPALA-7191: don't call srand() at random times .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-4848: Add WIDTH BUCKET() function

2018-06-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. ( http://gerrit.cloudera.org:8080/6023 ) Change subject: IMPALA-4848: Add WIDTH_BUCKET() function .. Patch Set 15: I already let Anuj know offline that it's ok to remove

[Impala-ASF-CR] IMPALA-5202: Disallow PREPARE:WAIT debug action

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10776 ) Change subject: IMPALA-5202: Disallow PREPARE:WAIT debug action .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-7046: introduce "global" debug actions

2018-06-20 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10690 ) Change subject: IMPALA-7046: introduce "global" debug_actions .. Patch Set 13: (1 comment)

[Impala-ASF-CR] IMPALA-7046: introduce "global" debug actions

2018-06-20 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/10690 ) Change subject: IMPALA-7046: introduce "global" debug_actions .. Patch Set 13: (1 comment)

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-20 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10778 ) Change subject: IMPALA-7191: don't call srand() at random times .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10778 ) Change subject: IMPALA-7191: don't call srand() at random times .. Patch Set 2: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10778 ) Change subject: IMPALA-7191: don't call srand() at random times .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-20 Thread Dan Hecht (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10778 to look at the new patch set (#2). Change subject: IMPALA-7191: don't call srand() at random times ..

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-20 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10778 ) Change subject: IMPALA-7191: don't call srand() at random times .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-20 Thread Dan Hecht (Code Review)
Dan Hecht has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10778 Change subject: IMPALA-7191: don't call srand() at random times .. IMPALA-7191: don't call srand() at random times Instead, call it from

[Impala-ASF-CR] IMPALA-5202: Disallow PREPARE:WAIT debug action

2018-06-20 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10776 ) Change subject: IMPALA-5202: Disallow PREPARE:WAIT debug action .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7151: Rework ephemeral port assignment for be tests

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10740 ) Change subject: IMPALA-7151: Rework ephemeral port assignment for be tests .. IMPALA-7151: Rework ephemeral port

[Impala-ASF-CR] IMPALA-7151: Rework ephemeral port assignment for be tests

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10740 ) Change subject: IMPALA-7151: Rework ephemeral port assignment for be tests .. Patch Set 5: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7187: Fix test group impersonation running inside Docker

2018-06-20 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10772 ) Change subject: IMPALA-7187: Fix test_group_impersonation running inside Docker .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-7187: Fix test group impersonation running inside Docker

2018-06-20 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10772 ) Change subject: IMPALA-7187: Fix test_group_impersonation running inside Docker .. Patch Set 3: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-7187: Fix test group impersonation running inside Docker

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10772 ) Change subject: IMPALA-7187: Fix test_group_impersonation running inside Docker .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-5202: Disallow PREPARE:WAIT debug action

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10776 ) Change subject: IMPALA-5202: Disallow PREPARE:WAIT debug action .. Patch Set 1: Code-Review+1 -- To view, visit

[Impala-ASF-CR] [WIP] Skip remaining exec rpcs if a backend has failed

2018-06-20 Thread Dan Hecht (Code Review)
Dan Hecht has abandoned this change. ( http://gerrit.cloudera.org:8080/10691 ) Change subject: [WIP] Skip remaining exec rpcs if a backend has failed .. Abandoned Oops, didn't mean to publish this. -- To view, visit

[Impala-ASF-CR] [WIP] Skip remaining exec rpcs if a backend has failed

2018-06-20 Thread Dan Hecht (Code Review)
Dan Hecht has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/10691 ) Change subject: [WIP] Skip remaining exec rpcs if a backend has failed .. [WIP] Skip remaining exec rpcs if a backend has failed Change-Id:

[Impala-ASF-CR] IMPALA-5202: Disallow PREPARE:WAIT debug action

2018-06-20 Thread Dan Hecht (Code Review)
Dan Hecht has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10776 Change subject: IMPALA-5202: Disallow PREPARE:WAIT debug action .. IMPALA-5202: Disallow PREPARE:WAIT debug action In order to simplify

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-20 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 2: (14 comments) I left some comments but I'm don't know how to

[Impala-ASF-CR] IMPALA-7150: fixes jni frame scope

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10746 ) Change subject: IMPALA-7150: fixes jni frame scope .. IMPALA-7150: fixes jni frame scope A previous change

[Impala-ASF-CR] IMPALA-7150: fixes jni frame scope

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10746 ) Change subject: IMPALA-7150: fixes jni frame scope .. Patch Set 3: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6119: Fix issue with multiple partitions sharing same location

2018-06-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/10543 ) Change subject: IMPALA-6119: Fix issue with multiple partitions sharing same location .. Patch Set 17: (2 comments)

[Impala-ASF-CR] [DOCS] Added another scenario when REFRESH is required

2018-06-20 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10773 ) Change subject: [DOCS] Added another scenario when REFRESH is required .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7046: introduce "global" debug actions

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10690 ) Change subject: IMPALA-7046: introduce "global" debug_actions .. IMPALA-7046: introduce "global" debug_actions The

[Impala-ASF-CR] IMPALA-7046: introduce "global" debug actions

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10690 ) Change subject: IMPALA-7046: introduce "global" debug_actions .. Patch Set 12: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7151: Rework ephemeral port assignment for be tests

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10740 ) Change subject: IMPALA-7151: Rework ephemeral port assignment for be tests .. Patch Set 4: Checked with Anuj directly... -- To

[Impala-ASF-CR] IMPALA-7151: Rework ephemeral port assignment for be tests

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10740 ) Change subject: IMPALA-7151: Rework ephemeral port assignment for be tests .. Patch Set 5: Build started:

[Impala-ASF-CR] IMPALA-7151: Rework ephemeral port assignment for be tests

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10740 ) Change subject: IMPALA-7151: Rework ephemeral port assignment for be tests .. Patch Set 5: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7150: fixes jni frame scope

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10746 ) Change subject: IMPALA-7150: fixes jni frame scope .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7150: fixes jni frame scope

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10746 ) Change subject: IMPALA-7150: fixes jni frame scope .. Patch Set 3: Build started:

[Impala-ASF-CR] [DOCS] Added another scenario when REFRESH is required

2018-06-20 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10773 Change subject: [DOCS] Added another scenario when REFRESH is required .. [DOCS] Added another scenario when REFRESH is required

[Impala-ASF-CR] IMPALA-4848: Add WIDTH BUCKET() function

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/6023 ) Change subject: IMPALA-4848: Add WIDTH_BUCKET() function .. Patch Set 15: Maybe Taras can comment on whether that precondition is

[Impala-ASF-CR] IMPALA-7121: Clean up partitionIds from HdfsTable

2018-06-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10654 ) Change subject: IMPALA-7121: Clean up partitionIds_ from HdfsTable .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-3307: Add support for IANA time-zone db

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9986 ) Change subject: IMPALA-3307: Add support for IANA time-zone db .. Patch Set 19: Thanks for running those tests! -- To view, visit

[Impala-ASF-CR] IMPALA-3307: Add support for IANA time-zone db

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9986 ) Change subject: IMPALA-3307: Add support for IANA time-zone db .. Patch Set 19: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7151: Rework ephemeral port assignment for be tests

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10740 ) Change subject: IMPALA-7151: Rework ephemeral port assignment for be tests .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-7151: Rework ephemeral port assignment for be tests

2018-06-20 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10740 ) Change subject: IMPALA-7151: Rework ephemeral port assignment for be tests .. Patch Set 4: Code-Review+2 (1 comment) LGTM. Please

[Impala-ASF-CR] IMPALA-7150: fixes jni frame scope

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10746 ) Change subject: IMPALA-7150: fixes jni frame scope .. Patch Set 2: No worries, we can continue to monitor builds since we do

[Impala-ASF-CR] IMPALA-7150: fixes jni frame scope

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10746 ) Change subject: IMPALA-7150: fixes jni frame scope .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7151: Rework ephemeral port assignment for be tests

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10740 ) Change subject: IMPALA-7151: Rework ephemeral port assignment for be tests .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-7121: Clean up partitionIds from HdfsTable

2018-06-20 Thread Gabor Kaszab (Code Review)
Hello Bharath Vissapragada, Zoltan Borok-Nagy, Sailesh Mukil, Todd Lipcon, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10654 to look at the new patch set (#4). Change subject: IMPALA-7121: Clean up partitionIds_ from HdfsTable

[Impala-ASF-CR] IMPALA-7151: Rework ephemeral port assignment for be tests

2018-06-20 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10740 ) Change subject: IMPALA-7151: Rework ephemeral port assignment for be tests .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-7150: fixes jni frame scope

2018-06-20 Thread Vuk Ercegovac (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10746 to look at the new patch set (#2). Change subject: IMPALA-7150: fixes jni frame scope .. IMPALA-7150: fixes jni

[Impala-ASF-CR] IMPALA-7150: fixes jni frame scope

2018-06-20 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10746 ) Change subject: IMPALA-7150: fixes jni frame scope .. Patch Set 1: (1 comment) I have not been able to repro-- this particular issue

[Impala-ASF-CR] IMPALA-7046: introduce "global" debug actions

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10690 ) Change subject: IMPALA-7046: introduce "global" debug_actions .. Patch Set 12: Build started:

[Impala-ASF-CR] IMPALA-7046: introduce "global" debug actions

2018-06-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10690 ) Change subject: IMPALA-7046: introduce "global" debug_actions .. Patch Set 12: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7046: introduce "global" debug actions

2018-06-20 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10690 ) Change subject: IMPALA-7046: introduce "global" debug_actions .. Patch Set 11: Code-Review+2 Carry +2, fix obvious missing reference

[Impala-ASF-CR] IMPALA-7046: introduce "global" debug actions

2018-06-20 Thread Dan Hecht (Code Review)
Hello Tim Armstrong, Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10690 to look at the new patch set (#11). Change subject: IMPALA-7046: introduce "global" debug_actions

[Impala-ASF-CR] IMPALA-7151: Rework ephemeral port assignment for be tests

2018-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10740 ) Change subject: IMPALA-7151: Rework ephemeral port assignment for be tests .. Patch Set 2: (8 comments)

[Impala-ASF-CR] IMPALA-7151: Rework ephemeral port assignment for be tests

2018-06-20 Thread Tim Armstrong (Code Review)
Hello Sailesh Mukil, anujphadke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10740 to look at the new patch set (#3). Change subject: IMPALA-7151: Rework ephemeral port assignment for be tests

[Impala-ASF-CR] IMPALA-3307: Add support for IANA time-zone db

2018-06-20 Thread Attila Jeges (Code Review)
Attila Jeges has posted comments on this change. ( http://gerrit.cloudera.org:8080/9986 ) Change subject: IMPALA-3307: Add support for IANA time-zone db .. Patch Set 19: > Are we ready to go ahead and merge? Would be good to

[Impala-ASF-CR] IMPALA-6119: Fix issue with multiple partitions sharing same location

2018-06-20 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has posted comments on this change. ( http://gerrit.cloudera.org:8080/10543 ) Change subject: IMPALA-6119: Fix issue with multiple partitions sharing same location .. Patch Set 16: Pls ignore the comments for

[Impala-ASF-CR] IMPALA-7187: Fix test group impersonation running inside Docker

2018-06-20 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10772 ) Change subject: IMPALA-7187: Fix test_group_impersonation running inside Docker .. Patch Set 3: I found the issue. So I decided to fix

[Impala-ASF-CR] IMPALA-7187: Fix test group impersonation running inside Docker

2018-06-20 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/10772 ) Change subject: IMPALA-7187: Fix test_group_impersonation running inside Docker .. IMPALA-7187: Fix test_group_impersonation running

[Impala-ASF-CR] IMPALA-6119: Fix issue with multiple partitions sharing same location

2018-06-20 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has posted comments on this change. ( http://gerrit.cloudera.org:8080/10543 ) Change subject: IMPALA-6119: Fix issue with multiple partitions sharing same location .. Patch Set 16: (11 comments)

[Impala-ASF-CR] IMPALA-6119: Fix issue with multiple partitions sharing same location

2018-06-20 Thread Gabor Kaszab (Code Review)
Hello Bharath Vissapragada, Michael Brown, Zoltan Borok-Nagy, Sailesh Mukil, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10543 to look at the new patch set (#16). Change subject: IMPALA-6119: Fix issue with multiple partitions sharing