Henry Robinson has posted comments on this change.

Change subject: IMPALA-4608: Fix fragment completion times for INSERTs
......................................................................


Patch Set 4: Code-Review+2

(1 comment)

Rebase, carry +2

http://gerrit.cloudera.org:8080/#/c/5418/2/be/src/runtime/coordinator.cc
File be/src/runtime/coordinator.cc:

Line 1673:       // Skip coordinator fragment, if one exists.
> can you leave a comment explaining what the special case is (i.e. skipped f
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3ce56f70d30c9e398b14b32520c815d87f81f893
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Henry Robinson <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-HasComments: Yes

Reply via email to