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

morningman pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-1.2-lts by this push:
     new 13588cad1b [fix](scanner scheduler) fix coredump of 
ScannerScheduler::_scanner_scan (#15199)
13588cad1b is described below

commit 13588cad1b471503a490b9a961f9a309bdaf5aa5
Author: TengJianPing <[email protected]>
AuthorDate: Wed Dec 21 15:44:47 2022 +0800

    [fix](scanner scheduler) fix coredump of ScannerScheduler::_scanner_scan 
(#15199)
    
    * [fix](scanner scheduler) fix coredump of ScannerScheduler::_scanner_scan
    
    * fix
---
 be/src/vec/exec/scan/scanner_scheduler.cpp         |  2 +-
 .../string_functions/test_split_part.groovy        | 30 ++++++++++++++++++++++
 2 files changed, 31 insertions(+), 1 deletion(-)

diff --git a/be/src/vec/exec/scan/scanner_scheduler.cpp 
b/be/src/vec/exec/scan/scanner_scheduler.cpp
index 62b3d022a8..0a8dd47932 100644
--- a/be/src/vec/exec/scan/scanner_scheduler.cpp
+++ b/be/src/vec/exec/scan/scanner_scheduler.cpp
@@ -235,7 +235,7 @@ void ScannerScheduler::_scanner_scan(ScannerScheduler* 
scheduler, ScannerContext
         VLOG_ROW << "VOlapScanNode input rows: " << block->rows() << ", eos: " 
<< eos;
         // The VFileScanner for external table may try to open not exist files,
         // Because FE file cache for external table may out of date.
-        if (!status.ok() && (typeid(*scanner) == 
typeid(doris::vectorized::VFileScanner) &&
+        if (!status.ok() && (typeid(*scanner) != 
typeid(doris::vectorized::VFileScanner) ||
                              !status.is_not_found())) {
             LOG(WARNING) << "Scan thread read VOlapScanner failed: " << 
status.to_string();
             // Add block ptr in blocks, prevent mem leak in read failed
diff --git 
a/regression-test/suites/query_p0/sql_functions/string_functions/test_split_part.groovy
 
b/regression-test/suites/query_p0/sql_functions/string_functions/test_split_part.groovy
new file mode 100644
index 0000000000..4369a1da1d
--- /dev/null
+++ 
b/regression-test/suites/query_p0/sql_functions/string_functions/test_split_part.groovy
@@ -0,0 +1,30 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.  See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership.  The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License.  You may obtain a copy of the License at
+//
+//   http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied.  See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+suite("test_split_part") {
+  test {
+    sql """
+      select
+          name
+      from
+          tpch_tiny_nation
+      where
+          split_part("bCKHDX07at", "5.7.37", cast(name as int)) is not null;
+    """
+    exception "errCode = 2, detailMessage = [RUNTIME_ERROR]Argument at index 3 
for function split_part must be constant"
+  }
+}
\ No newline at end of file


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to