[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-30 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 11: > From the discussion today: > > Points agreed on: > - Collapse Executors and states,

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-30 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 11: >From the discussion today: Points agreed on: - Collapse Executors and states, i.e. collaps

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-30 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 11: (20 comments) http://gerrit.cloudera.org:8080/#/c/3817/11/be/src/runtime/fragment-instance-exe

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-29 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 11: (43 comments) this needs a follow-on discussion regarding the existing class hierarchy (a

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-29 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 11: > (2 comments) > > As discussed in person earlier, we decided the best way forward > woul

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-28 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 11: (2 comments) As discussed in person earlier, we decided the best way forward would be to

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-28 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 10: Does this most recent patch compile? If not, how far away are you from getting it working?

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-24 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded a new patch set (#11). Change subject: IMPALA-4014: Introduce query-wide execution state. .. IMPALA-4014: Introduce query-wide execution state. This patch is a header preview of a query wide execution

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-24 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 10: (52 comments) http://gerrit.cloudera.org:8080/#/c/3817/10/be/src/runtime/fragment-instance-e

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-24 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 10: i'll do the next round once sailesh has addressed henry's comments. -- To view, visit ht

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-24 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/3817/10/be/src/runtime/fragment-instance-

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-23 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 10: (51 comments) http://gerrit.cloudera.org:8080/#/c/3817/10/be/src/runtime/fragment-instance-

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-23 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded a new patch set (#10). Change subject: IMPALA-4014: Introduce query-wide execution state. .. IMPALA-4014: Introduce query-wide execution state. This patch is a header preview of a query wide execution

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-08-23 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/3817/7/be/src/service/query-exec-mgr.cc File