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
 > would be to split this into header changes + implementation (the
 > header changes will allow Lars to start working on the per-query
 > rpc independently).
 > 
 > We also decided to introduce parallel headers to the existing ones
 > (so we'll have FragmentInstanceState and FragmentExecState).

I'm currently stuck with trying to solve a crash for IMPALA-3610 and was 
dealing with an escalation over the weekend. Seeing as how we want to get in 
3610 for this release, I will focus on that for today and try to get back to 
this later tonight.

-- 
To view, visit http://gerrit.cloudera.org:8080/3817
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I892091d6401acb2ea91ccb1623af54c6f9635e6c
Gerrit-PatchSet: 11
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com>
Gerrit-Reviewer: David Knupp <dkn...@cloudera.com>
Gerrit-Reviewer: Henry Robinson <he...@cloudera.com>
Gerrit-Reviewer: Lars Volker <l...@cloudera.com>
Gerrit-Reviewer: Marcel Kornacker <mar...@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com>
Gerrit-HasComments: No

Reply via email to