[incubator-doris] branch master updated: Fix the core of get_next in exchange node (#2505)

[zh...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

zhaoc pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 49b8097  Fix the core of get_next in exchange node (#2505)
49b8097 is described below

commit 49b809749512a10bd1bf54fe8854ba164f4ea730
Author: EmmyMiao87 <52...@qq.com>
AuthorDate: Thu Dec 19 16:40:33 2019 +0800

    Fix the core of get_next in exchange node (#2505)
    
    The _input_batch hasn't been initialized in exchange node.
    The undefined behavior will cause that the BE wants to get the capacity of input_batch before BE initialize it.
    The issue is #2504
---
 be/src/exec/exchange_node.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/be/src/exec/exchange_node.h b/be/src/exec/exchange_node.h
index 3a334b3..18df782 100644
--- a/be/src/exec/exchange_node.h
+++ b/be/src/exec/exchange_node.h
@@ -85,7 +85,7 @@ private:
     // Only valid if _is_merging is false. (If _is_merging is true, GetNext() is
     // delegated to the receiver). Owned by the stream receiver.
     // boost::scoped_ptr<RowBatch> _input_batch;
-    RowBatch* _input_batch;
+    RowBatch* _input_batch = nullptr;
 
     // Next row to copy from _input_batch. For non-merging exchanges, _input_batch
     // is retrieved directly from the sender queue in the stream recvr, and rows from
@@ -119,4 +119,3 @@ private:
 };
 
 #endif
-


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org