[Impala-ASF-CR] IMPALA-8766: Change cloud dependencies to use hadoop-cloud-storage

2019-07-18 Thread Joe McDonnell (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13872 to look at the new patch set (#2). Change subject: IMPALA-8766: Change cloud dependencies to use hadoop-cloud-storage

[Impala-ASF-CR] IMPALA-8766: Change cloud dependencies to use hadoop-cloud-storage

2019-07-18 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13872 ) Change subject: IMPALA-8766: Change cloud dependencies to use hadoop-cloud-storage .. Patch Set 2: Nexus needs https. -- To view, v

[Impala-ASF-CR] IMPALA-8176: Convert simple backend tests to the unified executable

2019-07-18 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/13515 ) Change subject: IMPALA-8176: Convert simple backend tests to the unified executable .. IMPALA-8176: Convert simple backend tests to the

[Impala-ASF-CR] IMPALA-8176: Convert simple backend tests to the unified executable

2019-07-19 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded a new patch set (#6). ( http://gerrit.cloudera.org:8080/13515 ) Change subject: IMPALA-8176: Convert simple backend tests to the unified executable .. IMPALA-8176: Convert simple backend tests to the

[Impala-ASF-CR] IMPALA-8176: Convert simple backend tests to the unified executable

2019-07-19 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13515 ) Change subject: IMPALA-8176: Convert simple backend tests to the unified executable .. Patch Set 6: (14 comments) http://gerrit.clou

[Impala-ASF-CR] IMPALA-8176: Convert simple backend tests to the unified executable

2019-07-19 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13515 ) Change subject: IMPALA-8176: Convert simple backend tests to the unified executable .. Patch Set 6: This changes the *Tests libraries

[Impala-ASF-CR] IMPALA-8176: Convert simple backend tests to the unified executable

2019-07-19 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13515 ) Change subject: IMPALA-8176: Convert simple backend tests to the unified executable .. IMPALA-8176: Convert simple backend tes

[Impala-ASF-CR] IMPALA-8766: Change cloud dependencies to use hadoop-cloud-storage

2019-07-19 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13872 ) Change subject: IMPALA-8766: Change cloud dependencies to use hadoop-cloud-storage .. Patch Set 3: The alternative to this is to add

[Impala-ASF-CR] IMPALA-8766: Change cloud dependencies to use hadoop-cloud-storage

2019-07-22 Thread Joe McDonnell (Code Review)
Hello Vihang Karajgaonkar, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13872 to look at the new patch set (#4). Change subject: IMPALA-8766: Change cloud dependencies to use hadoop-cloud-storage ...

[Impala-ASF-CR] IMPALA-8766: Change cloud dependencies to use hadoop-cloud-storage

2019-07-24 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13872 ) Change subject: IMPALA-8766: Change cloud dependencies to use hadoop-cloud-storage .. IMPALA-8766: Change cloud dependencies t

[Impala-ASF-CR] IMPALA-8785: give debug docker images a different name

2019-07-24 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13905 ) Change subject: IMPALA-8785: give debug docker images a different name .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/

[Impala-ASF-CR] IMPALA-8785: give debug docker images a different name

2019-07-24 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13905 ) Change subject: IMPALA-8785: give debug docker images a different name .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/

[Impala-ASF-CR] IMPALA-8766: Restrict hadoop-cloud-storage to USE CDP HIVE=true

2019-07-26 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13929 Change subject: IMPALA-8766: Restrict hadoop-cloud-storage to USE_CDP_HIVE=true .. IMPALA-8766: Restrict hadoop-cloud-storage to USE_CDP

[Impala-ASF-CR] IMPALA-8785: give debug docker images a different name

2019-07-30 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13905 ) Change subject: IMPALA-8785: give debug docker images a different name .. Patch Set 9: (3 comments) This makes sense. A couple small

[Impala-ASF-CR] IMPALA-8785: give debug docker images a different name

2019-07-30 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13905 ) Change subject: IMPALA-8785: give debug docker images a different name .. Patch Set 11: Code-Review+2 This looks good to me -- To vi

[Impala-ASF-CR] IMPALA-8820: fix start-impala-cluster catalogd startup

2019-07-31 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13971 ) Change subject: IMPALA-8820: fix start-impala-cluster catalogd startup .. Patch Set 2: Code-Review+2 This makes sense to me -- To vi

[Impala-ASF-CR] IMPALA-8516: Fix the sha512sum check for the Maven download

2019-07-31 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13876 ) Change subject: IMPALA-8516: Fix the sha512sum check for the Maven download .. Patch Set 1: This makes sense to me. Does this have any

[Impala-ASF-CR] IMPALA-4551: Limit the size of SQL statements

2019-08-05 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14012 Change subject: IMPALA-4551: Limit the size of SQL statements .. IMPALA-4551: Limit the size of SQL statements Various BI tools generat

[Impala-ASF-CR] IMPALA-4551: Limit the size of SQL statements

2019-08-05 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14012 ) Change subject: IMPALA-4551: Limit the size of SQL statements .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/14012/

[Impala-ASF-CR] IMPALA-8816: reduce custom cluster test runtime in core

2019-08-06 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13967 ) Change subject: IMPALA-8816: reduce custom cluster test runtime in core .. Patch Set 6: (2 comments) This looks good to me, a couple

[Impala-ASF-CR] IMPALA-8816: reduce custom cluster test runtime in core

2019-08-06 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13967 ) Change subject: IMPALA-8816: reduce custom cluster test runtime in core .. Patch Set 8: (1 comment) One final nit http://gerrit.clou

[Impala-ASF-CR] IMPALA-8766: Exorcise hadoop-cloud-storage + HWX Nexus

2019-08-06 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14024 Change subject: IMPALA-8766: Exorcise hadoop-cloud-storage + HWX Nexus .. IMPALA-8766: Exorcise hadoop-cloud-storage + HWX Nexus Previo

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-06 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14026 Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters .. IMPALA-8685,IMPALA-8677: Use consistent schedu

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-06 Thread Joe McDonnell (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14026 to look at the new patch set (#2). Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters .

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-06 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14026 ) Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters .. Patch Set 1: (1 comment) http://gerrit.clou

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-06 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14026 ) Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters .. Patch Set 2: I'm still thinking through the

[Impala-ASF-CR] IMPALA-8766: Undo hadoop-cloud-storage + HWX Nexus

2019-08-08 Thread Joe McDonnell (Code Review)
Hello Vihang Karajgaonkar, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14024 to look at the new patch set (#2). Change subject: IMPALA-8766: Undo hadoop-cloud-storage + HWX Nexus

[Impala-ASF-CR] IMPALA-4551: Limit the size of SQL statements

2019-08-08 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14012 ) Change subject: IMPALA-4551: Limit the size of SQL statements .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/14012/1

[Impala-ASF-CR] IMPALA-4551: Limit the size of SQL statements

2019-08-09 Thread Joe McDonnell (Code Review)
Hello Bharath Vissapragada, Michael Ho, Quanlong Huang, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14012 to look at the new patch set (#2). Change subject: IMPALA-4551: Limit the size of SQL statements

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-09 Thread Joe McDonnell (Code Review)
Hello Michael Ho, Lars Volker, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14026 to look at the new patch set (#3). Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

[Impala-ASF-CR] IMPALA-8766: Undo hadoop-cloud-storage + HWX Nexus

2019-08-10 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14024 ) Change subject: IMPALA-8766: Undo hadoop-cloud-storage + HWX Nexus .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/14

[Impala-ASF-CR] IMPALA-8766: Undo hadoop-cloud-storage + HWX Nexus

2019-08-10 Thread Joe McDonnell (Code Review)
Hello Vihang Karajgaonkar, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14024 to look at the new patch set (#3). Change subject: IMPALA-8766: Undo hadoop-cloud-storage + HWX Nexus

[Impala-ASF-CR] Revert "build: use thin static archives"

2019-08-11 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13798 ) Change subject: Revert "build: use thin static archives" .. Patch Set 2: > What was the rationale for this revert? I don't see a refer

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-12 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14026 ) Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters .. Patch Set 3: (3 comments) Working on a new

[Impala-ASF-CR] IMPALA-4551: Limit the size of SQL statements

2019-08-12 Thread Joe McDonnell (Code Review)
Hello Bharath Vissapragada, Michael Ho, Quanlong Huang, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14012 to look at the new patch set (#3). Change subject: IMPALA-4551: Limit the size of SQL statements

[Impala-ASF-CR] IMPALA-4551: Limit the size of SQL statements

2019-08-12 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14012 ) Change subject: IMPALA-4551: Limit the size of SQL statements .. Patch Set 2: (11 comments) http://gerrit.cloudera.org:8080/#/c/14012

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-12 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14026 ) Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters .. Patch Set 3: (3 comments) http://gerrit.clo

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-12 Thread Joe McDonnell (Code Review)
Hello Michael Ho, Lars Volker, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14026 to look at the new patch set (#4). Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

[Impala-ASF-CR] IMPALA-4551: Limit the size of SQL statements

2019-08-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14012 ) Change subject: IMPALA-4551: Limit the size of SQL statements .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/14012/3

[Impala-ASF-CR] IMPALA-4551: Limit the size of SQL statements

2019-08-13 Thread Joe McDonnell (Code Review)
Hello Bharath Vissapragada, Michael Ho, Quanlong Huang, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14012 to look at the new patch set (#4). Change subject: IMPALA-4551: Limit the size of SQL statements

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-13 Thread Joe McDonnell (Code Review)
Hello Michael Ho, Lars Volker, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14026 to look at the new patch set (#6). Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14026 ) Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters .. Patch Set 6: Code-Review+1 Carry +1 -- To

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14026 ) Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters .. Patch Set 5: (1 comment) http://gerrit.clou

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14026 ) Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters .. Patch Set 7: Code-Review+1 Carry +1 -- To

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-13 Thread Joe McDonnell (Code Review)
Hello Michael Ho, Lars Volker, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14026 to look at the new patch set (#8). Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14026 ) Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters .. Patch Set 7: (1 comment) http://gerrit.clou

[Impala-ASF-CR] IMPALA-4551: Limit the size of SQL statements

2019-08-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14012 ) Change subject: IMPALA-4551: Limit the size of SQL statements .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/14012/5

[Impala-ASF-CR] IMPALA-4551: Limit the size of SQL statements

2019-08-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14012 ) Change subject: IMPALA-4551: Limit the size of SQL statements .. Patch Set 6: Code-Review+2 Carry +2 -- To view, visit http://gerrit

[Impala-ASF-CR] IMPALA-4551: Limit the size of SQL statements

2019-08-13 Thread Joe McDonnell (Code Review)
Hello Bharath Vissapragada, Michael Ho, Quanlong Huang, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14012 to look at the new patch set (#6). Change subject: IMPALA-4551: Limit the size of SQL statements

[Impala-ASF-CR] [WIP] IMPALA-8821: Use RECOVER PARTITIONS in dataload to get partition metadata

2019-08-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14060 Change subject: [WIP] IMPALA-8821: Use RECOVER PARTITIONS in dataload to get partition metadata .. [WIP] IMPALA-8821: Use RECOVER PARTI

[Impala-ASF-CR] IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters

2019-08-14 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14026 ) Change subject: IMPALA-8685,IMPALA-8677: Use consistent scheduling for small clusters .. IMPALA-8685,IMPALA-8677: Use consiste

[Impala-ASF-CR] [WIP] Exclude hive-shim-0.20

2019-08-14 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14065 Change subject: [WIP] Exclude hive-shim-0.20 .. [WIP] Exclude hive-shim-0.20 Change-Id: Iccecd279759a966551e4451e31a5382fd55baa48 --- M

[Impala-ASF-CR] [WIP] Exclude hive-shim-0.20

2019-08-14 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14065 ) Change subject: [WIP] Exclude hive-shim-0.20 .. Patch Set 1: Not for review -- To view, visit http://gerrit.cloudera.org:8080/14065

[Impala-ASF-CR] [WIP] IMPALA-8821: Use RECOVER PARTITIONS in dataload to get partition metadata

2019-08-14 Thread Joe McDonnell (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14060 to look at the new patch set (#2). Change subject: [WIP] IMPALA-8821: Use RECOVER PARTITIONS in dataload to get partition metadata ...

[Impala-ASF-CR] IMPALA-9444: Fix URL for postgresql jar download

2020-03-02 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15337 Change subject: IMPALA-9444: Fix URL for postgresql jar download .. IMPALA-9444: Fix URL for postgresql jar download The current URL us

[Impala-ASF-CR] IMPALA-9444: Fix URL for postgresql jar download

2020-03-02 Thread Joe McDonnell (Code Review)
Hello Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15337 to look at the new patch set (#2). Change subject: IMPALA-9444: Fix URL for postgresql jar download

[Impala-ASF-CR] IMPALA-9444: Fix URL for postgresql jar download

2020-03-02 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15337 ) Change subject: IMPALA-9444: Fix URL for postgresql jar download .. Patch Set 2: Code-Review+2 (1 comment) Carry +2 http://gerrit.cl

[Impala-ASF-CR] IMPALA-9444: Fix URL for postgresql jar download

2020-03-02 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15337 ) Change subject: IMPALA-9444: Fix URL for postgresql jar download .. IMPALA-9444: Fix URL for postgresql jar download The curre

[Impala-ASF-CR] IMPALA-9472,IMPALA-9473: Add per-partition metrics for data cache

2020-03-06 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15382 Change subject: IMPALA-9472,IMPALA-9473: Add per-partition metrics for data cache .. IMPALA-9472,IMPALA-9473: Add per-partition metrics

[Impala-ASF-CR] IMPALA-9472,IMPALA-9473: Add per-partition metrics for data cache

2020-03-06 Thread Joe McDonnell (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15382 to look at the new patch set (#2). Change subject: IMPALA-9472,IMPALA-9473: Add per-partition metrics for data cache .

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-13 Thread Joe McDonnell (Code Review)
Hello Thomas Tauber-Marshall, Sahil Takiar, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15306 to look at the new patch set (#10). Change subject: IMPALA-8690: Add LIRS cache eviction algorithm ..

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15306 ) Change subject: IMPALA-8690: Add LIRS cache eviction algorithm .. Patch Set 9: (22 comments) http://gerrit.cloudera.org:8080/#/c/1530

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-13 Thread Joe McDonnell (Code Review)
Hello Thomas Tauber-Marshall, Sahil Takiar, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15306 to look at the new patch set (#13). Change subject: IMPALA-8690: Add LIRS cache eviction algorithm ..

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15306 ) Change subject: IMPALA-8690: Add LIRS cache eviction algorithm .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/15306

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-13 Thread Joe McDonnell (Code Review)
Hello Thomas Tauber-Marshall, Sahil Takiar, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15306 to look at the new patch set (#14). Change subject: IMPALA-8690: Add LIRS cache eviction algorithm ..

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15306 ) Change subject: IMPALA-8690: Add LIRS cache eviction algorithm .. Patch Set 13: (1 comment) http://gerrit.cloudera.org:8080/#/c/15306

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-13 Thread Joe McDonnell (Code Review)
Hello Thomas Tauber-Marshall, Sahil Takiar, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15306 to look at the new patch set (#15). Change subject: IMPALA-8690: Add LIRS cache eviction algorithm ..

[Impala-ASF-CR] Update version to 4.0.0-SNAPSHOT

2020-03-16 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15449 Change subject: Update version to 4.0.0-SNAPSHOT .. Update version to 4.0.0-SNAPSHOT Change-Id: I4fddb36b09f68d7a98950ecb1f4179f089a634

[Impala-ASF-CR](asf-site) IMPALA-9462: Point to downloads.apache.org for current release

2020-03-16 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15443 ) Change subject: IMPALA-9462: Point to downloads.apache.org for current release .. Patch Set 1: I was thinking I would update this when

[Impala-ASF-CR] IMPALA-9029: [DOCS] Impala 3.4 Release Notes

2020-03-16 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15451 ) Change subject: IMPALA-9029: [DOCS] Impala 3.4 Release Notes .. Patch Set 1: Two things: 1. Can we add a line to impala_new_features.x

[Impala-ASF-CR] IMPALA-9467: [DOCS] live progress enabled by default in interactive mode

2020-03-17 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15442 ) Change subject: IMPALA-9467: [DOCS] live_progress enabled by default in interactive mode .. Patch Set 2: Code-Review+2 -- To view, v

[Impala-ASF-CR] IMPALA-9467: [DOCS] live progress enabled by default in interactive mode

2020-03-17 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15442 ) Change subject: IMPALA-9467: [DOCS] live_progress enabled by default in interactive mode .. IMPALA-9467: [DOCS] live_progress

[Impala-ASF-CR] IMPALA-9029: [DOCS] Impala 3.4 Release Notes

2020-03-17 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15451 ) Change subject: IMPALA-9029: [DOCS] Impala 3.4 Release Notes .. Patch Set 1: You can abandon this change, now that it is addressed in

[Impala-ASF-CR] IMPALA-9029: [DOCS] Impala 3.4 Release Notes

2020-03-17 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14863 ) Change subject: IMPALA-9029: [DOCS] Impala 3.4 Release Notes .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.o

[Impala-ASF-CR] IMPALA-9029: [DOCS] Impala 3.4 Release Notes

2020-03-17 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14863 ) Change subject: IMPALA-9029: [DOCS] Impala 3.4 Release Notes .. IMPALA-9029: [DOCS] Impala 3.4 Release Notes -Added broadcast_

[Impala-ASF-CR] IMPALA-9517: [DOCS] Document broadcast bytes limit query option

2020-03-17 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15457 ) Change subject: IMPALA-9517: [DOCS] Document broadcast_bytes_limit query option .. Patch Set 2: Can you rebase this change to the late

[Impala-ASF-CR] IMPALA-9517: [DOCS] Document broadcast bytes limit query option

2020-03-17 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15457 ) Change subject: IMPALA-9517: [DOCS] Document broadcast_bytes_limit query option .. IMPALA-9517: [DOCS] Document broadcast_bytes

[Impala-ASF-CR] IMPALA-9517: [DOCS] Document broadcast bytes limit query option

2020-03-17 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15457 ) Change subject: IMPALA-9517: [DOCS] Document broadcast_bytes_limit query option .. Patch Set 5: Code-Review+2 -- To view, visit http:

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-18 Thread Joe McDonnell (Code Review)
Hello Thomas Tauber-Marshall, Sahil Takiar, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15306 to look at the new patch set (#16). Change subject: IMPALA-8690: Add LIRS cache eviction algorithm ..

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-18 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15306 ) Change subject: IMPALA-8690: Add LIRS cache eviction algorithm .. Patch Set 15: (6 comments) http://gerrit.cloudera.org:8080/#/c/1530

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-18 Thread Joe McDonnell (Code Review)
Hello Thomas Tauber-Marshall, Sahil Takiar, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15306 to look at the new patch set (#18). Change subject: IMPALA-8690: Add LIRS cache eviction algorithm ..

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-18 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15306 ) Change subject: IMPALA-8690: Add LIRS cache eviction algorithm .. Patch Set 17: (11 comments) http://gerrit.cloudera.org:8080/#/c/153

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-18 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15306 ) Change subject: IMPALA-8690: Add LIRS cache eviction algorithm .. Patch Set 17: (1 comment) http://gerrit.cloudera.org:8080/#/c/15306

[Impala-ASF-CR] IMPALA-8005: Randomize partitioning exchanges.

2020-03-19 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15497 ) Change subject: IMPALA-8005: Randomize partitioning exchanges. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/15497/

[Impala-ASF-CR] IMPALA-9191 (part 1): Allow Impala to run tests without Sentry

2020-03-20 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15505 ) Change subject: IMPALA-9191 (part 1): Allow Impala to run tests without Sentry .. Patch Set 4: (1 comment) This all looks good to me.

[Impala-ASF-CR] IMPALA-8533: Impala daemon crash on sort

2020-03-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15473 ) Change subject: IMPALA-8533: Impala daemon crash on sort .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8

[Impala-ASF-CR] Add timestamp to maven logging options.

2020-03-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15537 ) Change subject: Add timestamp to maven logging options. .. Patch Set 1: (4 comments) Please verify that https://github.com/apache/im

[Impala-ASF-CR] IMPALA-8533: Impala daemon crash on sort

2020-03-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15473 ) Change subject: IMPALA-8533: Impala daemon crash on sort .. Patch Set 6: Failure in precommit looks unrelated (one is IMPALA-9547, ano

[Impala-ASF-CR] IMPALA-9373: more tactical IWYU fixes

2020-03-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15248 ) Change subject: IMPALA-9373: more tactical IWYU fixes .. Patch Set 8: (3 comments) This makes sense to me. Thanks for putting this to

[Impala-ASF-CR] IMPALA-9107: Add timestamp to maven logging options.

2020-03-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15537 ) Change subject: IMPALA-9107: Add timestamp to maven logging options. .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c

[Impala-ASF-CR] IMPALA-9107: Add timestamp to maven logging options.

2020-03-24 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15537 ) Change subject: IMPALA-9107: Add timestamp to maven logging options. .. Patch Set 3: This looks good to me. I'm going to run gerrit-ve

[Impala-ASF-CR] IMPALA-9373: more tactical IWYU fixes

2020-03-24 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15248 ) Change subject: IMPALA-9373: more tactical IWYU fixes .. Patch Set 10: Code-Review+2 This looks good to me. -- To view, visit http:/

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-24 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15306 ) Change subject: IMPALA-8690: Add LIRS cache eviction algorithm .. Patch Set 18: (3 comments) http://gerrit.cloudera.org:8080/#/c/1530

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-24 Thread Joe McDonnell (Code Review)
Hello Thomas Tauber-Marshall, Sahil Takiar, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15306 to look at the new patch set (#19). Change subject: IMPALA-8690: Add LIRS cache eviction algorithm ..

[Impala-ASF-CR] IMPALA-8690: Add LIRS cache eviction algorithm

2020-03-24 Thread Joe McDonnell (Code Review)
Hello Thomas Tauber-Marshall, Sahil Takiar, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15306 to look at the new patch set (#20). Change subject: IMPALA-8690: Add LIRS cache eviction algorithm ..

[Impala-ASF-CR] IMPALA-9107: Add timestamp to maven logging options.

2020-03-25 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15537 ) Change subject: IMPALA-9107: Add timestamp to maven logging options. .. IMPALA-9107: Add timestamp to maven logging options. W

[Impala-ASF-CR] IMPALA-9472,IMPALA-9473: Add per-partition metrics for data cache

2020-03-25 Thread Joe McDonnell (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15382 to look at the new patch set (#3). Change subject: IMPALA-9472,IMPALA-9473: Add per-partition metrics for data cache .

[Impala-ASF-CR] IMPALA-9472,IMPALA-9473: Add per-partition metrics for data cache

2020-03-25 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15382 ) Change subject: IMPALA-9472,IMPALA-9473: Add per-partition metrics for data cache .. Patch Set 3: Just stacked this on top of the lat

[Impala-ASF-CR] IMPALA-8005: Randomize partitioning exchanges.

2020-03-25 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/15497 ) Change subject: IMPALA-8005: Randomize partitioning exchanges. .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/15497/

[Impala-ASF-CR] IMPALA-9549: Handle catalogd startup delays when using local catalog

2020-03-25 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15561 Change subject: IMPALA-9549: Handle catalogd startup delays when using local catalog .. IMPALA-9549: Handle catalogd startup delays whe

<    2   3   4   5   6   7   8   9   10   11   >