[bugfix]fix TableFunctionNode memory leak (#9853)

This commit is contained in:
Pxl
2022-05-31 19:20:22 +08:00
committed by GitHub
parent ec5027a00a
commit d34d631519
3 changed files with 4 additions and 3 deletions

View File

@ -377,6 +377,7 @@ Status TableFunctionNode::close(RuntimeState* state) {
}
RETURN_IF_ERROR(exec_debug_action(TExecNodePhase::CLOSE));
Expr::close(_fn_ctxs, state);
vectorized::VExpr::close(_vfn_ctxs, state);
if (_num_rows_filtered_counter != nullptr) {
COUNTER_SET(_num_rows_filtered_counter, static_cast<int64_t>(_num_rows_filtered));

View File

@ -418,7 +418,7 @@ public:
// follow ZSTD official example
// https://github.com/facebook/zstd/blob/dev/examples/streaming_compression.c
Status compress(const std::vector<Slice>& inputs, Slice* output) const {
Status compress(const std::vector<Slice>& inputs, Slice* output) const override {
if (!ctx_c) return Status::InvalidArgument("compression context NOT initialized");
// reset ctx to start new compress session
@ -479,7 +479,7 @@ public:
// follow ZSTD official example
// https://github.com/facebook/zstd/blob/dev/examples/streaming_decompression.c
Status decompress(const Slice& input, Slice* output) const {
Status decompress(const Slice& input, Slice* output) const override {
if (!ctx_d) return Status::InvalidArgument("decompression context NOT initialized");
// reset ctx to start a new decompress session

View File

@ -67,7 +67,7 @@ private:
Status _open_next_reader();
Status _next_arrow_batch();
Status _init_arrow_batch_if_necessary();
Status _init_src_block();
Status _init_src_block() override;
Status _append_batch_to_src_block(Block* block);
Status _cast_src_block(Block* block);