Michael Ho has posted comments on this change.

Change subject: IMPALA-2885: ScannerContext::Stream objects should be owned by 
ScannerContext
......................................................................


Patch Set 4:

More testing revealed that hdfs-scanners cache the handle to the Stream object. 
Text scanner may reference it after it has been destroyed. The new patch sets 
the cached handle to NULL in AddFinalRowBatch() and fixes the text scanner to 
not reference the cached handle of the stream after it has been freed.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic5440d414ecc0ca19676c553275aeb85231d6045
Gerrit-PatchSet: 4
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Michael Ho <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Michael Ho <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: No

Reply via email to