[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Michael Ho (Code Review)
Michael Ho has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. IMPALA-6323 Allow constant analytic window expressions.

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 15: Code-Review+2 -- To view, visit http://gerrit.c

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 15: Verified+1 -- To view, visit http://

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 15: Build started: https://jenkins.impala

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 15: Build Successful https://jenkins.imp

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Michal Ostrowski (Code Review)
Hello Michael Ho, Tim Armstrong, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11556 to look at the new patch set (#15). Change subject: IMPALA-6323 Allow constant analytic window expressions. .

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 14: Michal, can you please look into the failures he

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 14: Verified-1 Build failed: https://jenk

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 13: Build Successful https://jenkins.imp

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 14: Code-Review+2 -- To view, visit http

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 14: Build started: https://jenkins.impala

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 13: Code-Review+2 -- To view, visit http://gerrit.c

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-05 Thread Michal Ostrowski (Code Review)
Hello Michael Ho, Tim Armstrong, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11556 to look at the new patch set (#13). Change subject: IMPALA-6323 Allow constant analytic window expressions. .

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 12: Verified-1 Build failed: https://jenk

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 12: Build started: https://jenkins.impala

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-02 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 12: Code-Review+2 -- To view, visit http://gerrit.c

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 12: Build Successful https://jenkins.imp

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-02 Thread Michal Ostrowski (Code Review)
Michal Ostrowski has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 12: This should be ready to go now... -- To

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-02 Thread Michal Ostrowski (Code Review)
Hello Michael Ho, Tim Armstrong, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11556 to look at the new patch set (#12). Change subject: IMPALA-6323 Allow constant analytic window expressions. .

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 11: Verified-1 Build failed: https://jenk

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 11: Build Successful https://jenkins.imp

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Michal Ostrowski (Code Review)
Hello Michael Ho, Tim Armstrong, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11556 to look at the new patch set (#11). Change subject: IMPALA-6323 Allow constant analytic window expressions. .

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 11: Build started: https://jenkins.impala

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 10: Build Successful https://jenkins.imp

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Michal Ostrowski (Code Review)
Hello Michael Ho, Tim Armstrong, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11556 to look at the new patch set (#10). Change subject: IMPALA-6323 Allow constant analytic window expressions. .

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 9: Build started: https://jenkins.impala.

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 8: Verified-1 Build failed: https://jenki

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 8: Build Successful https://jenkins.impa

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 8: Build started: https://jenkins.impala.

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Michal Ostrowski (Code Review)
Hello Michael Ho, Tim Armstrong, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11556 to look at the new patch set (#8). Change subject: IMPALA-6323 Allow constant analytic window expressions. ..

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-11-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 7: Verified-1 Build failed: https://jenki

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-31 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 7: Build Successful https://jenkins.impa

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-31 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 7: Build started: https://jenkins.impala.

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-31 Thread Michal Ostrowski (Code Review)
Hello Michael Ho, Tim Armstrong, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11556 to look at the new patch set (#7). Change subject: IMPALA-6323 Allow constant analytic window expressions. ..

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-25 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 6: Should there also be an end-to-end test for this

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-25 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 6: https://jenkins.impala.io/job/ubuntu-16.04-fro

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-25 Thread Michal Ostrowski (Code Review)
Michal Ostrowski has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 6: > Patch Set 6: > > The test failures look l

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 6: The test failures look legit and related to th

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 6: Verified-1 Build failed: https://jenki

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 6: Build Successful https://jenkins.impa

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 6: Build started: https://jenkins.impala.

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-24 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-24 Thread Michal Ostrowski (Code Review)
Michal Ostrowski has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:80

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-24 Thread Michal Ostrowski (Code Review)
Hello Michael Ho, Tim Armstrong, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11556 to look at the new patch set (#6). Change subject: IMPALA-6323 Allow constant analytic window expressions. ..

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-24 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 5: Code-Review+2 LGTM once my two style comments

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-24 Thread Michal Ostrowski (Code Review)
Michal Ostrowski has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 5: (1 comment) > Patch Set 5: > > (1 comment)

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-22 Thread Michal Ostrowski (Code Review)
Michal Ostrowski has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:808

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 5: (3 comments) A few suggestions but otherwise

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 5: Build Successful https://jenkins.impa

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-12 Thread Michal Ostrowski (Code Review)
Hello Tim Armstrong, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11556 to look at the new patch set (#5). Change subject: IMPALA-6323 Allow constant analytic window expressions. ..

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11556 ) Change subject: IMPALA-6323 Allow constant analytic window expressions. .. Patch Set 4: Build Failed https://jenkins.impala.i

[Impala-ASF-CR] IMPALA-6323 Allow constant analytic window expressions.

2018-10-12 Thread Michal Ostrowski (Code Review)
Hello Tim Armstrong, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11556 to look at the new patch set (#4). Change subject: IMPALA-6323 Allow constant analytic window expressions. ..