[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5631 To

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. Patch Set 5: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/187/ --

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-12 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5631 To unsubscribe,

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-12 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. Patch Set 5: (1 comment)

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-12 Thread Thomas Tauber-Marshall (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5631 to look at the new patch set (#5). Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException ..

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-12 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. Patch Set 4: (2 comments)

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-12 Thread Thomas Tauber-Marshall (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5631 to look at the new patch set (#4). Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException ..

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-12 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. Patch Set 3: (2 comments)

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. Patch Set 3: Code-Review+1 LGTM, will let someone else approve -- To view, visit

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. IMPALA-4716: Expr rewrite causes IllegalStateException The DECODE constructor in CaseExpr uses

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/5631/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. IMPALA-4716: Expr rewrite causes IllegalStateException The DECODE constructor in CaseExpr uses

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/5631/1//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-4716: Expr rewrite causes IllegalStateException

2017-01-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/5631 Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException .. IMPALA-4716: Expr rewrite causes