Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-2885: ScannerContext::Stream objects should be owned by ScannerContext ......................................................................
IMPALA-2885: ScannerContext::Stream objects should be owned by ScannerContext Each scanner context has at least one stream which corresponds to a scan range. For parquet scanner, there can be multiple streams. These Stream objects are stored in the RuntimeState's object pool even though they have the same life span of the scanner threads. This change makes ScannerContext the owner of the Stream objects so they will be freed when ScannerContext is destroyed. Change-Id: Ic5440d414ecc0ca19676c553275aeb85231d6045 Reviewed-on: http://gerrit.cloudera.org:8080/3590 Reviewed-by: Michael Ho <[email protected]> Tested-by: Internal Jenkins --- M be/src/exec/hdfs-scanner.cc M be/src/exec/hdfs-text-scanner.cc M be/src/exec/scanner-context.cc M be/src/exec/scanner-context.h 4 files changed, 11 insertions(+), 8 deletions(-) Approvals: Michael Ho: Looks good to me, approved Internal Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3590 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic5440d414ecc0ca19676c553275aeb85231d6045 Gerrit-PatchSet: 8 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Michael Ho <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
