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

zhangstar333 pushed a commit to branch branch-2.0
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-2.0 by this push:
     new 22bf7548dc5 [cherry-pick](scan)scanner could eos early when reached 
limit (#36535) (#36738)
22bf7548dc5 is described below

commit 22bf7548dc5477b6d37a501e468bbbb71382819b
Author: zhangstar333 <87313068+zhangstar...@users.noreply.github.com>
AuthorDate: Wed Jun 26 14:14:52 2024 +0800

    [cherry-pick](scan)scanner could eos early when reached limit (#36535) 
(#36738)
    
    ## Proposed changes
    cherry-pick from master #36535
---
 be/src/vec/exec/scan/new_es_scan_node.cpp   | 4 ++--
 be/src/vec/exec/scan/new_file_scan_node.cpp | 7 +++----
 be/src/vec/exec/scan/new_jdbc_scan_node.cpp | 5 ++---
 be/src/vec/exec/scan/new_odbc_scan_node.cpp | 2 +-
 be/src/vec/exec/scan/new_olap_scan_node.cpp | 8 ++++----
 be/src/vec/exec/scan/vmeta_scan_node.cpp    | 5 ++---
 6 files changed, 14 insertions(+), 17 deletions(-)

diff --git a/be/src/vec/exec/scan/new_es_scan_node.cpp 
b/be/src/vec/exec/scan/new_es_scan_node.cpp
index 4704f5eb8c6..ab0b565a27a 100644
--- a/be/src/vec/exec/scan/new_es_scan_node.cpp
+++ b/be/src/vec/exec/scan/new_es_scan_node.cpp
@@ -165,8 +165,8 @@ Status 
NewEsScanNode::_init_scanners(std::list<VScannerSPtr>* scanners) {
                 properties, _column_names, _docvalue_context, &doc_value_mode);
 
         std::shared_ptr<NewEsScanner> scanner = NewEsScanner::create_shared(
-                _state, this, _limit_per_scanner, _tuple_id, properties, 
_docvalue_context,
-                doc_value_mode, _state->runtime_profile());
+                _state, this, _limit, _tuple_id, properties, 
_docvalue_context, doc_value_mode,
+                _state->runtime_profile());
 
         RETURN_IF_ERROR(scanner->prepare(_state, _conjuncts));
         scanners->push_back(scanner);
diff --git a/be/src/vec/exec/scan/new_file_scan_node.cpp 
b/be/src/vec/exec/scan/new_file_scan_node.cpp
index da33538b8c3..2cc34f65f18 100644
--- a/be/src/vec/exec/scan/new_file_scan_node.cpp
+++ b/be/src/vec/exec/scan/new_file_scan_node.cpp
@@ -121,10 +121,9 @@ Status 
NewFileScanNode::_init_scanners(std::list<VScannerSPtr>* scanners) {
             std::min<size_t>(config::doris_scanner_thread_pool_thread_num, 
_scan_ranges.size());
     _kv_cache.reset(new ShardedKVCache(shard_num));
     for (auto& scan_range : _scan_ranges) {
-        std::unique_ptr<VFileScanner> scanner =
-                VFileScanner::create_unique(_state, this, _limit_per_scanner,
-                                            
scan_range.scan_range.ext_scan_range.file_scan_range,
-                                            runtime_profile(), 
_kv_cache.get());
+        std::unique_ptr<VFileScanner> scanner = VFileScanner::create_unique(
+                _state, this, _limit, 
scan_range.scan_range.ext_scan_range.file_scan_range,
+                runtime_profile(), _kv_cache.get());
         RETURN_IF_ERROR(
                 scanner->prepare(_conjuncts, &_colname_to_value_range, 
&_colname_to_slot_id));
         scanners->push_back(std::move(scanner));
diff --git a/be/src/vec/exec/scan/new_jdbc_scan_node.cpp 
b/be/src/vec/exec/scan/new_jdbc_scan_node.cpp
index f8219b4337e..deb3f636814 100644
--- a/be/src/vec/exec/scan/new_jdbc_scan_node.cpp
+++ b/be/src/vec/exec/scan/new_jdbc_scan_node.cpp
@@ -65,9 +65,8 @@ Status 
NewJdbcScanNode::_init_scanners(std::list<VScannerSPtr>* scanners) {
     if (_eos == true) {
         return Status::OK();
     }
-    std::unique_ptr<NewJdbcScanner> scanner =
-            NewJdbcScanner::create_unique(_state, this, _limit_per_scanner, 
_tuple_id,
-                                          _query_string, _table_type, 
_state->runtime_profile());
+    std::unique_ptr<NewJdbcScanner> scanner = NewJdbcScanner::create_unique(
+            _state, this, _limit, _tuple_id, _query_string, _table_type, 
_state->runtime_profile());
     RETURN_IF_ERROR(scanner->prepare(_state, _conjuncts));
     scanners->push_back(std::move(scanner));
     return Status::OK();
diff --git a/be/src/vec/exec/scan/new_odbc_scan_node.cpp 
b/be/src/vec/exec/scan/new_odbc_scan_node.cpp
index eafad165968..c6361b49bab 100644
--- a/be/src/vec/exec/scan/new_odbc_scan_node.cpp
+++ b/be/src/vec/exec/scan/new_odbc_scan_node.cpp
@@ -66,7 +66,7 @@ Status 
NewOdbcScanNode::_init_scanners(std::list<VScannerSPtr>* scanners) {
         return Status::OK();
     }
     std::shared_ptr<NewOdbcScanner> scanner = NewOdbcScanner::create_shared(
-            _state, this, _limit_per_scanner, _odbc_scan_node, 
_state->runtime_profile());
+            _state, this, _limit, _odbc_scan_node, _state->runtime_profile());
     RETURN_IF_ERROR(scanner->prepare(_state, _conjuncts));
     scanners->push_back(scanner);
     return Status::OK();
diff --git a/be/src/vec/exec/scan/new_olap_scan_node.cpp 
b/be/src/vec/exec/scan/new_olap_scan_node.cpp
index 3e55823f0ec..61147cc77a8 100644
--- a/be/src/vec/exec/scan/new_olap_scan_node.cpp
+++ b/be/src/vec/exec/scan/new_olap_scan_node.cpp
@@ -513,8 +513,8 @@ Status 
NewOlapScanNode::_init_scanners(std::list<VScannerSPtr>* scanners) {
                                      const std::vector<OlapScanRange*>& 
key_ranges,
                                      TabletReader::ReadSource read_source) {
             std::shared_ptr<NewOlapScanner> scanner = 
NewOlapScanner::create_shared(
-                    _state, this, _limit_per_scanner, 
_olap_scan_node.is_preaggregation, scan_range,
-                    key_ranges, std::move(read_source), 
_scanner_profile.get());
+                    _state, this, _limit, _olap_scan_node.is_preaggregation, 
scan_range, key_ranges,
+                    std::move(read_source), _scanner_profile.get());
 
             RETURN_IF_ERROR(scanner->prepare(_state, _conjuncts));
             scanner->set_compound_filters(_compound_filters);
@@ -605,8 +605,8 @@ Status 
NewOlapScanNode::_init_scanners(std::list<VScannerSPtr>* scanners) {
         auto build_new_scanner = [&](const TPaloScanRange& scan_range,
                                      const std::vector<OlapScanRange*>& 
key_ranges) {
             std::shared_ptr<NewOlapScanner> scanner = 
NewOlapScanner::create_shared(
-                    _state, this, _limit_per_scanner, 
_olap_scan_node.is_preaggregation, scan_range,
-                    key_ranges, _scanner_profile.get());
+                    _state, this, _limit, _olap_scan_node.is_preaggregation, 
scan_range, key_ranges,
+                    _scanner_profile.get());
 
             RETURN_IF_ERROR(scanner->prepare(_state, _conjuncts));
             scanner->set_compound_filters(_compound_filters);
diff --git a/be/src/vec/exec/scan/vmeta_scan_node.cpp 
b/be/src/vec/exec/scan/vmeta_scan_node.cpp
index 5ba559466ef..23aecf447af 100644
--- a/be/src/vec/exec/scan/vmeta_scan_node.cpp
+++ b/be/src/vec/exec/scan/vmeta_scan_node.cpp
@@ -68,9 +68,8 @@ Status VMetaScanNode::_init_scanners(std::list<VScannerSPtr>* 
scanners) {
     }
 
     for (auto& scan_range : _scan_ranges) {
-        std::shared_ptr<VMetaScanner> scanner =
-                VMetaScanner::create_shared(_state, this, _tuple_id, 
scan_range, _limit_per_scanner,
-                                            runtime_profile(), _user_identity);
+        std::shared_ptr<VMetaScanner> scanner = VMetaScanner::create_shared(
+                _state, this, _tuple_id, scan_range, _limit, 
runtime_profile(), _user_identity);
         RETURN_IF_ERROR(scanner->prepare(_state, _conjuncts));
         scanners->push_back(scanner);
     }


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

Reply via email to