[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 8: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7351 To

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5612: join inversion should factor in parallelism .. IMPALA-5612: join inversion should factor in parallelism The join inversion

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 8: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1117/ -- To

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 8: Code-Review+2 Missed updating a couple of planner tests -- To view, visit

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-22 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Bharath Vissapragada, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7351 to look at the new patch set (#8). Change subject: IMPALA-5612: join inversion should factor in parallelism

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 7: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 7: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1116/ -- To

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 7: Code-Review+2 rebase -- To view, visit http://gerrit.cloudera.org:8080/7351 To

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-21 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7351 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 5: (7 comments)

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-21 Thread Tim Armstrong (Code Review)
Hello Bharath Vissapragada, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7351 to look at the new patch set (#6). Change subject: IMPALA-5612: join inversion should factor in parallelism

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-21 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 5: (7 comments) Nice!

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-08-14 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 5: Code-Review+1 Carry +1 -- To view, visit http://gerrit.cloudera.org:8080/7351 To

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-07-11 Thread Tim Armstrong (Code Review)
Hello Bharath Vissapragada, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7351 to look at the new patch set (#4). Change subject: IMPALA-5612: join inversion should factor in parallelism

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-07-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 4: Code-Review+1 carry +1 -- To view, visit http://gerrit.cloudera.org:8080/7351 To

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-07-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/7351/3//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-07-10 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 3: Code-Review+1 (4 comments)

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-07-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 2: (6 comments)

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-07-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#3). Change subject: IMPALA-5612: join inversion should factor in parallelism .. IMPALA-5612: join inversion should factor in parallelism The join inversion optimisation did not

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-07-07 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 2: (5 comments) I have some minor comments, the patch looks ok to me otherwise.

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-07-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). Change subject: IMPALA-5612: join inversion should factor in parallelism .. IMPALA-5612: join inversion should factor in parallelism The join inversion optimisation did not

[Impala-ASF-CR] IMPALA-5612: join inversion should factor in parallelism

2017-07-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 1: (3 comments)