[Impala-ASF-CR] IMPALA-7234: Improve memory estimates produced by the Planner

2018-07-23 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/11001 ) Change subject: IMPALA-7234: Improve memory estimates produced by the Planner .. Patch Set 3: (3 comments)

[Impala-ASF-CR] IMPALA-5031: Fix undefined behavior: memset NULL

2018-07-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10948 ) Change subject: IMPALA-5031: Fix undefined behavior: memset NULL .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-5031: Fix undefined behavior: memset NULL

2018-07-23 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/10948 ) Change subject: IMPALA-5031: Fix undefined behavior: memset NULL .. Patch Set 4: Code-Review+2 carry Tim's +2 -- To view, visit

[Impala-ASF-CR] IMPALA-6299: Modify IRBuilder to use LLVM's CPU features

2018-07-23 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10979 ) Change subject: IMPALA-6299: Modify IRBuilder to use LLVM's CPU features .. Patch Set 5: Code-Review+1 (3 comments)

[Impala-ASF-CR] Test gerrit trigger 2

2018-07-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11012 ) Change subject: Test gerrit trigger 2 .. Patch Set 3: Build Failed

[Impala-ASF-CR] Test gerrit trigger 2

2018-07-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11012 ) Change subject: Test gerrit trigger 2 .. Patch Set 2: Build Failed

[Impala-ASF-CR] Test gerrit trigger with docs change

2018-07-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11013 Change subject: Test gerrit trigger with docs change .. Test gerrit trigger with docs change Change-Id:

[Impala-ASF-CR] Test gerrit trigger 2

2018-07-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11012 ) Change subject: Test gerrit trigger 2 .. Patch Set 2: Build Started

[Impala-ASF-CR] Test gerrit trigger 2

2018-07-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11012 ) Change subject: Test gerrit trigger 2 .. Patch Set 4: Build Started

[Impala-ASF-CR] Test gerrit trigger 2

2018-07-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11012 ) Change subject: Test gerrit trigger 2 .. Patch Set 3: Build Started

[Impala-ASF-CR] Test gerrit trigger 2

2018-07-23 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11012 to look at the new patch set (#4). Change subject: Test gerrit trigger 2 .. Test gerrit trigger 2

[Impala-ASF-CR] Test gerrit trigger 2

2018-07-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/11012 ) Change subject: Test gerrit trigger 2 .. Test gerrit trigger 2 Change-Id: I87f87032ca041f7ac68d782a48fe7bab84edbdf9 --- M

[Impala-ASF-CR] Test gerrit trigger not disabled

2018-07-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/11011 ) Change subject: Test gerrit trigger not disabled .. Test gerrit trigger not disabled Change-Id:

[Impala-ASF-CR] IMPALA-7212: Removes --use krpc flag and remove old DataStream services

2018-07-23 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10835 ) Change subject: IMPALA-7212: Removes --use_krpc flag and remove old DataStream services .. Patch Set 5: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-7212: Removes --use krpc flag and remove old DataStream services

2018-07-23 Thread Michael Ho (Code Review)
Hello Sailesh Mukil, Joe McDonnell, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10835 to look at the new patch set (#5). Change subject: IMPALA-7212: Removes --use_krpc flag and remove old DataStream services

[Impala-ASF-CR] IMPALA-7212: Removes --use krpc flag and remove old DataStream services

2018-07-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/10835 ) Change subject: IMPALA-7212: Removes --use_krpc flag and remove old DataStream services .. Patch Set 5: Code-Review+1 Rebased. Carry

[Impala-ASF-CR] IMPALA-7209: Disallow self referencing in ALTER VIEW statements

2018-07-23 Thread Pooja Nilangekar (Code Review)
Pooja Nilangekar has posted comments on this change. ( http://gerrit.cloudera.org:8080/10908 ) Change subject: IMPALA-7209: Disallow self referencing in ALTER VIEW statements .. Patch Set 8: (7 comments)

[Impala-ASF-CR] IMPALA-7209: Disallow self referencing in ALTER VIEW statements

2018-07-23 Thread Pooja Nilangekar (Code Review)
Pooja Nilangekar has uploaded a new patch set (#8). ( http://gerrit.cloudera.org:8080/10908 ) Change subject: IMPALA-7209: Disallow self referencing in ALTER VIEW statements .. IMPALA-7209: Disallow self referencing in ALTER

[Impala-ASF-CR] IMPALA-7173: [DOCS] Added check options in the load balancer examples

2018-07-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11006 ) Change subject: IMPALA-7173: [DOCS] Added check options in the load balancer examples .. Patch Set 1: Verified+1 -- To

[Impala-ASF-CR] IMPALA-7173: [DOCS] Added check options in the load balancer examples

2018-07-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11006 ) Change subject: IMPALA-7173: [DOCS] Added check options in the load balancer examples .. IMPALA-7173: [DOCS] Added

[native-toolchain-CR] Bump Kudu version to 5211897

2018-07-23 Thread Thomas Marshall (Code Review)
Thomas Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11003 ) Change subject: Bump Kudu version to 5211897 .. Bump Kudu version to 5211897 Testing: - Ran a full toolchain build.

[Impala-ASF-CR] IMPALA-7173: [DOCS] Added check options in the load balancer examples

2018-07-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11006 ) Change subject: IMPALA-7173: [DOCS] Added check options in the load balancer examples .. Patch Set 1: Build started:

[Impala-ASF-CR] Test gerrit trigger

2018-07-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11010 ) Change subject: Test gerrit trigger .. Patch Set 3: Verified-1 Build Failed

[native-toolchain-CR] Bump Kudu version to 5211897

2018-07-23 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/11003 ) Change subject: Bump Kudu version to 5211897 .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/11003 To

[Impala-ASF-CR] IMPALA-7173: [DOCS] Added check options in the load balancer examples

2018-07-23 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/11006 ) Change subject: IMPALA-7173: [DOCS] Added check options in the load balancer examples .. Patch Set 1: Code-Review+2 -- To view,

[Impala-ASF-CR] Test gerrit trigger disabled

2018-07-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11011 Change subject: Test gerrit trigger disabled .. Test gerrit trigger disabled Change-Id: I17e7c82ed127c8d94dc6e9425e94d82bd5dcd4fd ---

[Impala-ASF-CR] Test gerrit trigger

2018-07-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11010 ) Change subject: Test gerrit trigger .. Patch Set 3: Build Started https://jenkins.impala.io/job/gerrit-code-review-checks/3/

[Impala-ASF-CR] Test gerrit trigger

2018-07-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/11010 ) Change subject: Test gerrit trigger .. Test gerrit trigger Change-Id: I52314e4b72f36d9bef0d9572fcaa00668f0c0097 --- A test.txt 1 file

[Impala-ASF-CR] Test gerrit trigger

2018-07-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/11010 ) Change subject: Test gerrit trigger .. Test gerrit trigger Change-Id: I52314e4b72f36d9bef0d9572fcaa00668f0c0097 --- M

[Impala-ASF-CR] Test gerrit trigger

2018-07-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11010 Change subject: Test gerrit trigger .. Test gerrit trigger Change-Id: I52314e4b72f36d9bef0d9572fcaa00668f0c0097 --- M

[Impala-ASF-CR] IMPALA-7202: Add width bucket() to the decimal fuzz test

2018-07-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10859 ) Change subject: IMPALA-7202: Add width_bucket() to the decimal fuzz test .. Patch Set 1: Code-Review+1 LGTM, just need to wait for

[Impala-ASF-CR] IMPALA-5031: Fix undefined behavior: memset NULL

2018-07-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10948 ) Change subject: IMPALA-5031: Fix undefined behavior: memset NULL .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-5826: [DOCS] Documented the IDLE SESSION TIMEOUT query option

2018-07-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11004 ) Change subject: IMPALA-5826: [DOCS] Documented the IDLE_SESSION_TIMEOUT query option .. Patch Set 1: Code-Review+1 I'll let Balasz

[Impala-ASF-CR] IMPALA-6490: Reconnect shell when remote restarts

2018-07-23 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/10992 ) Change subject: IMPALA-6490: Reconnect shell when remote restarts .. Patch Set 5: (5 comments)

[Impala-ASF-CR] IMPALA-7209: Disallow self referencing in ALTER VIEW statements

2018-07-23 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/10908 ) Change subject: IMPALA-7209: Disallow self referencing in ALTER VIEW statements .. Patch Set 7: (7 comments) Looks pretty

<    1   2