[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. IMPALA-4707: fix use-after-free in QueryExecMgr The bug is that the QueryState was

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5615 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 6: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/160/ -- To view,

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-10 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5615 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-10 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 6: I had to make a non-trivial change to fix a bug, would be good if reviewers could take another

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-10 Thread Tim Armstrong (Code Review)
Hello Marcel Kornacker, Impala Public Jenkins, Jim Apple, Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5615 to look at the new patch set (#6). Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-09 Thread Tim Armstrong (Code Review)
Hello Marcel Kornacker, Impala Public Jenkins, Jim Apple, Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5615 to look at the new patch set (#4). Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 3: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/150/ --

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 3: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/150/ -- To view,

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 3: Code-Review+2 (1 comment) Carry +2

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-06 Thread Tim Armstrong (Code Review)
Hello Marcel Kornacker, Impala Public Jenkins, Jim Apple, Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5615 to look at the new patch set (#3). Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 2: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/149/ --

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-05 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5615/2/be/src/runtime/query-exec-mgr.cc File

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/149/ -- To view,

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-05 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5615 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 2: Carry the +1 -- To view, visit http://gerrit.cloudera.org:8080/5615 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-05 Thread Tim Armstrong (Code Review)
Hello Jim Apple, Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5615 to look at the new patch set (#2). Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr ..

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/5615/1/be/src/runtime/query-exec-mgr.cc File

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-05 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 1: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5615/1/be/src/runtime/query-exec-mgr.cc File

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-05 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 1: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-05 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5615/1/be/src/runtime/query-exec-mgr.cc File

[Impala-ASF-CR] IMPALA-4707: fix use-after-free in QueryExecMgr

2017-01-05 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4707: fix use-after-free in QueryExecMgr .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5615/1/be/src/runtime/query-exec-mgr.cc File