impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Ho (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-6262: Always initialize runtime profile for DataSink
Date Wed, 06 Dec 2017 09:03:14 GMT
Michael Ho has uploaded a new patch set (#2). ( )

Change subject: IMPALA-6262: Always initialize runtime profile for DataSink

IMPALA-6262: Always initialize runtime profile for DataSink

This change moves the creation of the runtime profile from DataSink::Prepare()
to the ctor of DataSink derived classes. This makes sure that DataSink::Close()
and other functions can access the profile even if the DataSink fails to initialize.

Testing done: Added a test case which triggers failure in the initialization of output
expressions in a HdfsTableSink. Impalad crashed consistently without the fix.

Change-Id: I2a683000ef180027b929dbebe78bc2a530a4767e
M be/src/exec/
M be/src/exec/data-sink.h
M be/src/exec/
M be/src/exec/hbase-table-sink.h
M be/src/exec/
M be/src/exec/hdfs-table-sink.h
M be/src/exec/
M be/src/exec/kudu-table-sink.h
M be/src/exec/
M be/src/exec/
M be/src/exec/partitioned-hash-join-builder.h
M be/src/exec/
M be/src/exec/plan-root-sink.h
M be/src/runtime/
M be/src/runtime/data-stream-sender.h
M be/src/runtime/
M be/src/runtime/
M be/src/runtime/krpc-data-stream-sender.h
A testdata/workloads/functional-query/queries/QueryTest/insert_bad_expr.test
M tests/query_test/
20 files changed, 79 insertions(+), 46 deletions(-)

  git pull ssh:// refs/changes/70/8770/2
To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2a683000ef180027b929dbebe78bc2a530a4767e
Gerrit-Change-Number: 8770
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Ho <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: Lars Volker <>
Gerrit-Reviewer: Sailesh Mukil <>
Gerrit-Reviewer: Tim Armstrong <>

  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message