diff --git a/be/src/exec/schema_scanner.cpp b/be/src/exec/schema_scanner.cpp index 39dd45163322ac..4b430f04289d04 100644 --- a/be/src/exec/schema_scanner.cpp +++ b/be/src/exec/schema_scanner.cpp @@ -124,7 +124,6 @@ Status SchemaScanner::get_next_block_async(RuntimeState* state) { } SCOPED_ATTACH_TASK(state); _async_thread_running = true; - _finish_dependency->block(); if (!_opened) { _data_block = vectorized::Block::create_unique(); _init_block(_data_block.get()); @@ -140,9 +139,6 @@ Status SchemaScanner::get_next_block_async(RuntimeState* state) { _eos = eos; _async_thread_running = false; _dependency->set_ready(); - if (eos) { - _finish_dependency->set_ready(); - } })); return Status::OK(); } diff --git a/be/src/exec/schema_scanner.h b/be/src/exec/schema_scanner.h index 440912bff1d729..6e7a229b7fd7b9 100644 --- a/be/src/exec/schema_scanner.h +++ b/be/src/exec/schema_scanner.h @@ -106,11 +106,7 @@ class SchemaScanner { // factory function static std::unique_ptr create(TSchemaTableType::type type); TSchemaTableType::type type() const { return _schema_table_type; } - void set_dependency(std::shared_ptr dep, - std::shared_ptr fin_dep) { - _dependency = dep; - _finish_dependency = fin_dep; - } + void set_dependency(std::shared_ptr dep) { _dependency = dep; } Status get_next_block_async(RuntimeState* state); protected: @@ -139,7 +135,6 @@ class SchemaScanner { RuntimeProfile::Counter* _fill_block_timer = nullptr; std::shared_ptr _dependency = nullptr; - std::shared_ptr _finish_dependency = nullptr; std::unique_ptr _data_block; AtomicStatus _scanner_status; diff --git a/be/src/pipeline/exec/schema_scan_operator.cpp b/be/src/pipeline/exec/schema_scan_operator.cpp index 006ecf8ad82e84..ddc2821cac14a1 100644 --- a/be/src/pipeline/exec/schema_scan_operator.cpp +++ b/be/src/pipeline/exec/schema_scan_operator.cpp @@ -48,7 +48,7 @@ Status SchemaScanLocalState::init(RuntimeState* state, LocalStateInfo& info) { // new one scanner _schema_scanner = SchemaScanner::create(schema_table->schema_table_type()); - _schema_scanner->set_dependency(_data_dependency, _finish_dependency); + _schema_scanner->set_dependency(_data_dependency); if (nullptr == _schema_scanner) { return Status::InternalError("schema scanner get nullptr pointer."); } @@ -266,9 +266,6 @@ Status SchemaScanOperatorX::get_block(RuntimeState* state, vectorized::Block* bl } while (block->rows() == 0 && !*eos); local_state.reached_limit(block, eos); - if (*eos) { - local_state._finish_dependency->set_always_ready(); - } return Status::OK(); } diff --git a/be/src/pipeline/exec/schema_scan_operator.h b/be/src/pipeline/exec/schema_scan_operator.h index 03cf422fbc52e6..c8ddf885e98a0f 100644 --- a/be/src/pipeline/exec/schema_scan_operator.h +++ b/be/src/pipeline/exec/schema_scan_operator.h @@ -36,9 +36,6 @@ class SchemaScanLocalState final : public PipelineXLocalState<> { SchemaScanLocalState(RuntimeState* state, OperatorXBase* parent) : PipelineXLocalState<>(state, parent) { - _finish_dependency = - std::make_shared(parent->operator_id(), parent->node_id(), - parent->get_name() + "_FINISH_DEPENDENCY", true); _data_dependency = std::make_shared(parent->operator_id(), parent->node_id(), parent->get_name() + "_DEPENDENCY", true); } @@ -48,7 +45,6 @@ class SchemaScanLocalState final : public PipelineXLocalState<> { Status open(RuntimeState* state) override; - Dependency* finishdependency() override { return _finish_dependency.get(); } std::vector dependencies() const override { return {_data_dependency.get()}; } private: @@ -57,7 +53,6 @@ class SchemaScanLocalState final : public PipelineXLocalState<> { SchemaScannerParam _scanner_param; std::unique_ptr _schema_scanner; - std::shared_ptr _finish_dependency; std::shared_ptr _data_dependency; };