[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7871 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5855: reserve enough memory for preaggs .. IMPALA-5855: reserve enough memory for preaggs The calculation in the planner failed to

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-30 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7871 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 6: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1176/ -- To view,

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 5: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/1175/ -- To

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-30 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7871 to look at the new patch set (#6). Change subject: IMPALA-5855: reserve enough memory for preaggs

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 5: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1175/ -- To view,

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-30 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 5: Code-Review+2 fix tests -- To view, visit http://gerrit.cloudera.org:8080/7871 To unsubscribe,

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 4: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/1172/ -- To

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-29 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1172/ -- To view,

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-29 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 4: Code-Review+2 rebase -- To view, visit http://gerrit.cloudera.org:8080/7871 To unsubscribe,

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-29 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7871 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-29 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#3). Change subject: IMPALA-5855: reserve enough memory for preaggs .. IMPALA-5855: reserve enough memory for preaggs The calculation in the planner failed to account for the

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-29 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 2: Fixed a couple of tests -- To view, visit http://gerrit.cloudera.org:8080/7871 To unsubscribe,

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-29 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 2: > I think you may need to update test admission-reject-min-reservation.test now > too, since

[Impala-ASF-CR] IMPALA-5855: reserve enough memory for preaggs

2017-08-29 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5855: reserve enough memory for preaggs .. Patch Set 2: I think you may need to update test admission-reject-min-reservation.test now too, since that