Alex Behm has uploaded a new patch set (#2). Change subject: IMPALA-3952: Clear scratch batch mem pool if Open() failed. ......................................................................
IMPALA-3952: Clear scratch batch mem pool if Open() failed. We used to be able to hit a DCHECK in HdfsParquetScanner::Close() when using the legacy aggs/joins if HdfsParquetScanner::Open() failed. With the legacy aggs/joins the tuple ptrs of the scratch batch are allocated from the scratch batch's mem pool, and if Open() failed we never freed or transferred the scratch batch's mem pool. Testing: I tested this patch together with the fix for IMPALA-3964 on core/hdfs with the legacy aggs and joins enabled. Change-Id: I55f32ed698a5b6fed8c28af1391aa07e1560e782 --- M be/src/exec/hdfs-parquet-scanner.cc 1 file changed, 7 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/53/3953/2 -- To view, visit http://gerrit.cloudera.org:8080/3953 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I55f32ed698a5b6fed8c28af1391aa07e1560e782 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Alex Behm <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]>
