[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. IMPALA-1575: Part 1: eagerly release query

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 16: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 16: Build started:

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 16: Code-Review+2 carry -- To view, visit

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-27 Thread Tim Armstrong (Code Review)
Hello Sailesh Mukil, Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8303 to look at the new patch set (#15). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 14: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-27 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 14: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 11: (3 comments)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-27 Thread Tim Armstrong (Code Review)
Hello Sailesh Mukil, Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8303 to look at the new patch set (#14). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-27 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 11: (3 comments)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 13: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-26 Thread Tim Armstrong (Code Review)
Hello Sailesh Mukil, Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8303 to look at the new patch set (#13). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-26 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 11: (8 comments)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-26 Thread Tim Armstrong (Code Review)
Hello Sailesh Mukil, Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8303 to look at the new patch set (#12). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-26 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 11: (7 comments)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-26 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 11: Code-Review+1 (1 comment) Thanks for the

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-26 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 10: (1 comment)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-26 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 10: (1 comment)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-26 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 11: Code-Review+1 (4 comments) Thanks for the

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-26 Thread Tim Armstrong (Code Review)
Hello Sailesh Mukil, Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8303 to look at the new patch set (#11). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-26 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 10: (4 comments) Just have a few minor

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 10: Code-Review+1 Carry Joe's +1 -- To view,

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 9: (2 comments)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-23 Thread Tim Armstrong (Code Review)
Hello Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8303 to look at the new patch set (#10). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 9: Code-Review+1 (1 comment) This makes sense

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 9: (1 comment)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-20 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 9: (1 comment) Making my way through this.

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-18 Thread Tim Armstrong (Code Review)
Hello Joe McDonnell, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8303 to look at the new patch set (#9). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources ..

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

2017-10-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#8). ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. IMPALA-1575: Part 1: eagerly release query exec resources