[Impala-ASF-CR] Fix formatting of HBaseScanNode explain output.

2020-04-17 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15749 Change subject: Fix formatting of HBaseScanNode explain output. .. Fix formatting of HBaseScanNode explain output. In the case with

[Impala-ASF-CR] IMPALA-9727: Fix HBaseScanNode explain formatting

2020-05-05 Thread Shant Hovsepian (Code Review)
Hello Quanlong Huang, Thomas Tauber-Marshall, Fang-Yu Rao, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15749 to look at the new patch set (#2). Change subject: IMPALA-9727: Fix HBaseScanNode explain formatting

[Impala-ASF-CR] IMPALA-9727: Fix HBaseScanNode explain formatting

2020-05-05 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/15749 ) Change subject: IMPALA-9727: Fix HBaseScanNode explain formatting .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-10099: Push down DISTINCT in Set operations

2020-08-30 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16350 ) Change subject: IMPALA-10099: Push down DISTINCT in Set operations .. Patch Set 4: > Patch Set 3: > > (1 comment) Good point. The

[Impala-ASF-CR] IMPALA-10099: Push down DISTINCT in Set operations

2020-08-30 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16350 to look at the new patch set (#4). Change subject: IMPALA-10099: Push down DISTINCT in Set operations

[Impala-ASF-CR] IMPALA-10064: Support constant propagation for eligible range predicates

2020-08-31 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16346 ) Change subject: IMPALA-10064: Support constant propagation for eligible range predicates .. Patch Set 9: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-31 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16349 ) Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs. .. Patch Set 12: > Patch Set 12: Verified-1 > > Build

[Impala-ASF-CR] IMPALA-9979: part 2: partitioned top-n

2020-09-10 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16242 ) Change subject: IMPALA-9979: part 2: partitioned top-n .. Patch Set 17: (6 comments)

[Impala-ASF-CR] IMPALA-9979: part 2: partitioned top-n

2020-09-11 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16242 ) Change subject: IMPALA-9979: part 2: partitioned top-n .. Patch Set 17: (1 comment)

[Impala-ASF-CR] IMPALA-8291: Show constraints in DESCRIBE FORMATTED

2020-09-12 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16428 ) Change subject: IMPALA-8291: Show constraints in DESCRIBE FORMATTED .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-10167: Docs typo for DEFAULT TRANSACTIONAL TYPE

2020-09-14 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16447 ) Change subject: IMPALA-10167: Docs typo for DEFAULT_TRANSACTIONAL_TYPE .. Patch Set 1: Small docs typo fix. -- To view, visit

[Impala-ASF-CR] IMPALA-10167: Docs typo for DEFAULT TRANSACTIONAL TYPE

2020-09-14 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16447 Change subject: IMPALA-10167: Docs typo for DEFAULT_TRANSACTIONAL_TYPE .. IMPALA-10167: Docs typo for DEFAULT_TRANSACTIONAL_TYPE Fix

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-31 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16349 ) Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs. .. Patch Set 10: (1 comment)

[Impala-ASF-CR] IMPALA-8291: Show constraints in DESCRIBE FORMATTED

2020-09-08 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16428 Change subject: IMPALA-8291: Show constraints in DESCRIBE FORMATTED .. IMPALA-8291: Show constraints in DESCRIBE FORMATTED Support

[Impala-ASF-CR] IMPALA-8291: Show constraints in DESCRIBE FORMATTED

2020-09-08 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16428 ) Change subject: IMPALA-8291: Show constraints in DESCRIBE FORMATTED .. Patch Set 1: Hi guys I tried to follow the standard of

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-31 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Qifan Chen, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16349 to look at the new patch set (#11). Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs.

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-31 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16349 ) Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs. .. Patch Set 11: (10 comments) Yeah I guess some of those

[Impala-ASF-CR] IMPALA-9974: Join elimination based on referential integrity.

2020-10-06 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16182 Change subject: IMPALA-9974: Join elimination based on referential integrity. .. IMPALA-9974: Join elimination based on referential

[Impala-ASF-CR] IMPALA-6671: [WIP] Skip locked tables from topic updates

2020-10-07 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16549 ) Change subject: IMPALA-6671: [WIP] Skip locked tables from topic updates .. Patch Set 1: (6 comments) Thanks for working on this

[Impala-ASF-CR] IMPALA-10262: RPM/DEB Packaging Support

2020-10-19 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16612 Change subject: IMPALA-10262: RPM/DEB Packaging Support .. IMPALA-10262: RPM/DEB Packaging Support cmake -DBUILD_PACKAGES=ON make

[Impala-ASF-CR] (WIP) IMPALA-10262: RPM/DEB Packaging Support

2020-10-19 Thread Shant Hovsepian (Code Review)
Hello Quanlong Huang, Grant Henke, Tim Armstrong, Joe McDonnell, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16612 to look at the new patch set (#2). Change subject: (WIP) IMPALA-10262: RPM/DEB Packaging Support

[Impala-ASF-CR] IMPALA-10262: RPM/DEB Packaging Support

2020-10-19 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16612 ) Change subject: IMPALA-10262: RPM/DEB Packaging Support .. Patch Set 1: Work in progress, just wanted to see if anyone has any

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-23 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Qifan Chen, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16349 to look at the new patch set (#3). Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs.

[Impala-ASF-CR] IMPALA-10099: Push down DISTINCT for SetOperations

2020-08-23 Thread Shant Hovsepian (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16350 to look at the new patch set (#2). Change subject: IMPALA-10099: Push down DISTINCT for SetOperations ..

[Impala-ASF-CR] IMPALA-10095: Include query plan tests for all of TPC-DS

2020-08-25 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16345 to look at the new patch set (#7). Change subject: IMPALA-10095: Include query plan tests for all of TPC-DS

[Impala-ASF-CR] IMPALA-10095: Include query plan tests for all of TPC-DS

2020-08-25 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16345 to look at the new patch set (#6). Change subject: IMPALA-10095: Include query plan tests for all of TPC-DS

[Impala-ASF-CR] IMPALA-10095: Include query plan tests for all of TPC-DS

2020-08-25 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16345 ) Change subject: IMPALA-10095: Include query plan tests for all of TPC-DS .. Patch Set 6: Too many parallel CR had some things sneak

[Impala-ASF-CR] IMPALA-10095: Include query plan tests for all of TPC-DS

2020-08-26 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16345 to look at the new patch set (#8). Change subject: IMPALA-10095: Include query plan tests for all of TPC-DS

[Impala-ASF-CR] IMPALA-10064: Support constant propagation for eligible range predicates

2020-08-28 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16346 ) Change subject: IMPALA-10064: Support constant propagation for eligible range predicates .. Patch Set 8: (5 comments)

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-28 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Qifan Chen, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16349 to look at the new patch set (#8). Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs.

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-28 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16349 ) Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs. .. Patch Set 8: This is getting a little uglier now. Turns

[Impala-ASF-CR] IMPALA-10110: bloom filter target fpp query option

2020-08-28 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16377 ) Change subject: IMPALA-10110: bloom filter target fpp query option .. Patch Set 5: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-28 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16349 ) Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs. .. Patch Set 8: (1 comment) > Patch Set 8: > > I wonder

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-29 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Qifan Chen, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16349 to look at the new patch set (#9). Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs.

[Impala-ASF-CR] IMPALA-10034: Add remaining TPC-DS queries to workload.

2020-08-19 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16280 ) Change subject: IMPALA-10034: Add remaining TPC-DS queries to workload. .. Patch Set 5: (2 comments) > Patch Set 4: > > Agree with

[Impala-ASF-CR] IMPALA-10034: Add remaining TPC-DS queries to workload.

2020-08-19 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Fang-Yu Rao, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16280 to look at the new patch set (#5). Change subject: IMPALA-10034: Add remaining TPC-DS queries to workload.

[Impala-ASF-CR] IMPALA-10034: Add remaining TPC-DS queries to workload.

2020-08-21 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Fang-Yu Rao, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16280 to look at the new patch set (#7). Change subject: IMPALA-10034: Add remaining TPC-DS queries to workload.

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-21 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Qifan Chen, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16349 to look at the new patch set (#2). Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs.

[Impala-ASF-CR] IMPALA-10099: Push down DISTINCT for SetOperations

2020-08-21 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16350 Change subject: IMPALA-10099: Push down DISTINCT for SetOperations .. IMPALA-10099: Push down DISTINCT for SetOperations Testing: *

[Impala-ASF-CR] IMPALA-10095: Include query plan tests for all of TPC-DS

2020-08-20 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16345 Change subject: IMPALA-10095: Include query plan tests for all of TPC-DS .. IMPALA-10095: Include query plan tests for all of TPC-DS

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-21 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16349 ) Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs. .. Patch Set 1: This is a bit of a lazy fix, handling only

[Impala-ASF-CR] IMPALA-7782: fix constant NOT IN subqueries that can return 0 rows

2020-08-19 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16338 ) Change subject: IMPALA-7782: fix constant NOT IN subqueries that can return 0 rows .. Patch Set 1: Code-Review+1 Nice catch. --

[Impala-ASF-CR] IMPALA-10034: Add remaining TPC-DS queries to workload.

2020-08-20 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Fang-Yu Rao, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16280 to look at the new patch set (#6). Change subject: IMPALA-10034: Add remaining TPC-DS queries to workload.

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-21 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16349 Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs. .. IMPALA-7310: Partial fix for NDV cardinality with NULLs.

[Impala-ASF-CR] IMPALA-10099: Push down DISTINCT in Set operations

2020-08-26 Thread Shant Hovsepian (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16350 to look at the new patch set (#3). Change subject: IMPALA-10099: Push down DISTINCT in Set operations ..

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-27 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Qifan Chen, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16349 to look at the new patch set (#5). Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs.

[Impala-ASF-CR] IMPALA-10046: Support constant propagation for eligible range predicates

2020-08-27 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16346 ) Change subject: IMPALA-10046: Support constant propagation for eligible range predicates .. Patch Set 7: (7 comments) Nice

[Impala-ASF-CR] IMPALA-10099: Push down DISTINCT in Set operations

2020-08-27 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16350 ) Change subject: IMPALA-10099: Push down DISTINCT in Set operations .. Patch Set 3: Small optimization for the set operation

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-27 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Qifan Chen, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16349 to look at the new patch set (#6). Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs.

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-27 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16349 ) Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs. .. Patch Set 6: (2 comments)

[Impala-ASF-CR] IMPALA-10110: allow setting bloom filter fpp

2020-08-27 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16377 ) Change subject: IMPALA-10110: allow setting bloom filter fpp .. Patch Set 4: Code-Review+1 (2 comments) Non critical suggestions.

[Impala-ASF-CR] IMPALA-10095: Include query plan tests for all of TPC-DS

2020-08-24 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16345 to look at the new patch set (#4). Change subject: IMPALA-10095: Include query plan tests for all of TPC-DS

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-24 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Qifan Chen, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16349 to look at the new patch set (#4). Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs.

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-24 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16349 ) Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs. .. Patch Set 4: (2 comments) > Patch Set 3: > > (1

[Impala-ASF-CR] IMPALA-10095: Include query plan tests for all of TPC-DS

2020-08-24 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16345 ) Change subject: IMPALA-10095: Include query plan tests for all of TPC-DS .. Patch Set 5: (2 comments)

[Impala-ASF-CR] IMPALA-10095: Include query plan tests for all of TPC-DS

2020-08-24 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16345 to look at the new patch set (#5). Change subject: IMPALA-10095: Include query plan tests for all of TPC-DS

[Impala-ASF-CR] IMPALA-7310: Partial fix for NDV cardinality with NULLs.

2020-08-29 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Qifan Chen, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16349 to look at the new patch set (#10). Change subject: IMPALA-7310: Partial fix for NDV cardinality with NULLs.

[Impala-ASF-CR] IMPALA-10112: Remove FpRateTooHigh() check for blom filter

2020-09-23 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16499 ) Change subject: IMPALA-10112: Remove FpRateTooHigh() check for blom filter .. Patch Set 1: (4 comments) Thanks for implementing

[Impala-ASF-CR] IMPALA-10185 Use bool stats for selectivity calculations.

2020-09-24 Thread Shant Hovsepian (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16462 to look at the new patch set (#2). Change subject: IMPALA-10185 Use bool stats for selectivity calculations.

[Impala-ASF-CR] IMPALA-10185 Use bool stats for selectivity calculations.

2020-09-24 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16462 ) Change subject: IMPALA-10185 Use bool stats for selectivity calculations. .. Patch Set 2: Just a small change nothing urgent. --

[Impala-ASF-CR] IMPALA-5022 part 1: Implement core functions of outer join simplification

2020-09-22 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16266 ) Change subject: IMPALA-5022 part 1: Implement core functions of outer join simplification .. Patch Set 22: Code-Review+1 Xianqing,

[Impala-ASF-CR] IMPALA-10112: Remove FpRateTooHigh() check for bloom filter

2020-09-25 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16499 ) Change subject: IMPALA-10112: Remove FpRateTooHigh() check for bloom filter .. Patch Set 2: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-8291: Show constraints in DESCRIBE FORMATTED

2020-09-14 Thread Shant Hovsepian (Code Review)
Hello Quanlong Huang, Vihang Karajgaonkar, David Rorke, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16428 to look at the new patch set (#2). Change subject: IMPALA-8291: Show constraints in DESCRIBE FORMATTED

[Impala-ASF-CR] IMPALA-9979: part 2: partitioned top-n

2020-09-14 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16242 ) Change subject: IMPALA-9979: part 2: partitioned top-n .. Patch Set 17: Code-Review+1 (2 comments) I don't feel as strongly about

[Impala-ASF-CR] IMPALA-7097 Print EC info in the query plan and profile

2020-10-26 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16587 ) Change subject: IMPALA-7097 Print EC info in the query plan and profile .. Patch Set 7: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-9943, IMPALA-4974: INTERSECT and EXCEPT DISTINCT Support.

2020-07-16 Thread Shant Hovsepian (Code Review)
Hello David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16123 to look at the new patch set (#6). Change subject: IMPALA-9943, IMPALA-4974: INTERSECT and EXCEPT DISTINCT Support.

[Impala-ASF-CR](asf-site) Add link to slack channel on community

2020-07-19 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16024 ) Change subject: Add link to slack channel on community .. Patch Set 1: Code-Review+1 > Patch Set 1: > > Could we just use

[Impala-ASF-CR] IMPALA-10034: Add remaining TPC-DS queries to workload.

2020-08-03 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16280 Change subject: IMPALA-10034: Add remaining TPC-DS queries to workload. .. IMPALA-10034: Add remaining TPC-DS queries to workload.

[Impala-ASF-CR] IMPALA-10034: Add remaining TPC-DS queries to workload.

2020-08-03 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16280 ) Change subject: IMPALA-10034: Add remaining TPC-DS queries to workload. .. Patch Set 1: Adding the missing tpc-ds queries. Easy

[Impala-ASF-CR] IMPALA-10034: Add remaining TPC-DS queries to workload.

2020-08-03 Thread Shant Hovsepian (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16280 to look at the new patch set (#2). Change subject: IMPALA-10034: Add remaining TPC-DS queries to workload.

[Impala-ASF-CR] IMPALA-10034: Add remaining TPC-DS queries to workload.

2020-08-06 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Fang-Yu Rao, David Rorke, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16280 to look at the new patch set (#4). Change subject: IMPALA-10034: Add remaining TPC-DS queries to workload.

[Impala-ASF-CR] IMPALA-5022: Outer join simplification

2020-08-04 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16266 ) Change subject: IMPALA-5022: Outer join simplification .. Patch Set 5: (10 comments) Hi Xianqing, thank you so much for this

[Impala-ASF-CR] IMPALA-10034: Add remaining TPC-DS queries to workload.

2020-08-07 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16280 ) Change subject: IMPALA-10034: Add remaining TPC-DS queries to workload. .. Patch Set 4: (2 comments)

[Impala-ASF-CR] IMPALA-9897: GROUPING SETS/CUBE/ROLLUP parsing and analysis

2020-06-30 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16112 ) Change subject: IMPALA-9897: GROUPING SETS/CUBE/ROLLUP parsing and analysis .. Patch Set 10: Code-Review+1 (2 comments) LGTM, nice

[Impala-ASF-CR] IMPALA-9917: grouping() and grouping id() support

2020-07-07 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16140 ) Change subject: IMPALA-9917: grouping() and grouping_id() support .. Patch Set 12: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-8954: Uncorrelated scalar subqueries in the select list

2020-07-04 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16007 ) Change subject: IMPALA-8954: Uncorrelated scalar subqueries in the select list .. Patch Set 8: (1 comment) > Patch Set 7:

[Impala-ASF-CR] IMPALA-8954: Uncorrelated scalar subqueries in the select list

2020-07-04 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Fang-Yu Rao, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16007 to look at the new patch set (#8). Change subject: IMPALA-8954: Uncorrelated scalar subqueries in the select list

[Impala-ASF-CR] IMPALA-9917: grouping() and grouping id() support

2020-07-06 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16140 ) Change subject: IMPALA-9917: grouping() and grouping_id() support .. Patch Set 7: (3 comments) Grouping Sets are usually useful

[Impala-ASF-CR] IMPALA-8954: Uncorrelated scalar subqueries in the select list

2020-07-04 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Fang-Yu Rao, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16007 to look at the new patch set (#9). Change subject: IMPALA-8954: Uncorrelated scalar subqueries in the select list

[Impala-ASF-CR] IMPALA-8954: Uncorrelated scalar subqueries in the select list

2020-07-04 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16007 ) Change subject: IMPALA-8954: Uncorrelated scalar subqueries in the select list .. Patch Set 8: (11 comments) Addressed some

[Impala-ASF-CR] IMPALA-8984: Uncorrelated scalar subqueries in the select list

2020-07-03 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Fang-Yu Rao, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16007 to look at the new patch set (#7). Change subject: IMPALA-8984: Uncorrelated scalar subqueries in the select list

[Impala-ASF-CR] IMPALA-8984: Uncorrelated scalar subqueries in the select list

2020-07-03 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16007 to look at the new patch set (#6). Change subject: IMPALA-8984: Uncorrelated scalar subqueries in the select list

[Impala-ASF-CR] IMPALA-8984: Uncorrelated scalar subqueries in the select list

2020-07-03 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16007 ) Change subject: IMPALA-8984: Uncorrelated scalar subqueries in the select list .. Patch Set 6: Fangu-Yu, FYI tweaked the reason

[Impala-ASF-CR] IMPALA-9897: GROUPING SETS/CUBE/ROLLUP parsing and analysis

2020-07-03 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16112 ) Change subject: IMPALA-9897: GROUPING SETS/CUBE/ROLLUP parsing and analysis .. Patch Set 13: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-9784, IMPALA-9905: Uncorrelated subqueries in HAVING.

2020-07-04 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16052 to look at the new patch set (#5). Change subject: IMPALA-9784, IMPALA-9905: Uncorrelated subqueries in HAVING.

[Impala-ASF-CR] IMPALA-9784, IMPALA-9905: Uncorrelated subqueries in HAVING.

2020-07-04 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16052 to look at the new patch set (#4). Change subject: IMPALA-9784, IMPALA-9905: Uncorrelated subqueries in HAVING.

[Impala-ASF-CR] IMPALA-9949: fix SELECT list subqueries with HAVING/LIMIT

2020-07-12 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16171 ) Change subject: IMPALA-9949: fix SELECT list subqueries with HAVING/LIMIT .. Patch Set 5: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-891: INTERSECT and EXCEPT Support

2020-07-01 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16123 Change subject: IMPALA-891: INTERSECT and EXCEPT Support .. IMPALA-891: INTERSECT and EXCEPT Support [WIP] Lots of code cleanup and

[Impala-ASF-CR] IMPALA-5226: handle single subquery in or predicate

2020-07-08 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16152 ) Change subject: IMPALA-5226: handle single subquery in or predicate .. Patch Set 6: Code-Review+1 (3 comments)

[Impala-ASF-CR] IMPALA-8984: Uncorrelated scalar subqueries in the select list

2020-06-25 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16007 to look at the new patch set (#4). Change subject: IMPALA-8984: Uncorrelated scalar subqueries in the select list

[Impala-ASF-CR] IMPALA-8984: Uncorrelated scalar subqueries in the select list

2020-06-25 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16007 ) Change subject: IMPALA-8984: Uncorrelated scalar subqueries in the select list .. Patch Set 4: (9 comments) Some more tests and

[Impala-ASF-CR] IMPALA-9784: Non correlated subqueries in HAVING.

2020-06-25 Thread Shant Hovsepian (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16052 to look at the new patch set (#2). Change subject: IMPALA-9784: Non correlated subqueries in HAVING. ..

[Impala-ASF-CR] IMPALA-9784: Non correlated subqueries in HAVING.

2020-06-25 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16052 ) Change subject: IMPALA-9784: Non correlated subqueries in HAVING. .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-7020: fix costing of non-trivial CAST expressions

2020-06-24 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16073 ) Change subject: IMPALA-7020: fix costing of non-trivial CAST expressions .. Patch Set 4: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-9866: Query plan debug page stuck in fetch and render loop.

2020-06-19 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16092 ) Change subject: IMPALA-9866: Query plan debug page stuck in fetch and render loop. .. Patch Set 1: (2 comments) Added a field to

[Impala-ASF-CR] IMPALA-9866: Query plan debug page stuck in fetch and render loop.

2020-06-19 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16092 ) Change subject: IMPALA-9866: Query plan debug page stuck in fetch and render loop. .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-9866: Query plan debug page stuck in fetch and render loop.

2020-06-19 Thread Shant Hovsepian (Code Review)
Hello Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16092 to look at the new patch set (#2). Change subject: IMPALA-9866: Query plan debug page stuck in fetch and render loop.

[Impala-ASF-CR] IMPALA-9866: Query plan debug page stuck in fetch and render loop.

2020-06-17 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16092 Change subject: IMPALA-9866: Query plan debug page stuck in fetch and render loop. .. IMPALA-9866: Query plan debug page stuck in

[Impala-ASF-CR] IMPALA-9784, IMPALA-9905: Uncorrelated subqueries in HAVING.

2020-06-28 Thread Shant Hovsepian (Code Review)
Hello Aman Sinha, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16052 to look at the new patch set (#3). Change subject: IMPALA-9784, IMPALA-9905: Uncorrelated subqueries in HAVING.

[Impala-ASF-CR] IMPALA-9784, IMPALA-9905: Uncorrelated subqueries in HAVING.

2020-06-28 Thread Shant Hovsepian (Code Review)
Shant Hovsepian has posted comments on this change. ( http://gerrit.cloudera.org:8080/16052 ) Change subject: IMPALA-9784, IMPALA-9905: Uncorrelated subqueries in HAVING. .. Patch Set 3: (4 comments) CR comments addressed,

  1   2   >