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

github-bot pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git


The following commit(s) were added to refs/heads/main by this push:
     new 49782435f2 Fix out-of-bounds access in SLT runner (#18562)
49782435f2 is described below

commit 49782435f2f6a77121b36da29eed1d9fb4ffbc4a
Author: theirix <[email protected]>
AuthorDate: Sun Nov 9 11:10:21 2025 +0000

    Fix out-of-bounds access in SLT runner (#18562)
    
    ## Which issue does this PR close?
    
    ## Rationale for this change
    
    A small fix for a rare case in SLT runner when it panics instead of
    printing result.
    
    ## What changes are included in this PR?
    
    Code change in sqllogictest
    
    ## Are these changes tested?
    
    Manual test
    
    ## Are there any user-facing changes?
    
    No
---
 datafusion/sqllogictest/src/util.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/datafusion/sqllogictest/src/util.rs 
b/datafusion/sqllogictest/src/util.rs
index 695fe463fa..2c3bd12d89 100644
--- a/datafusion/sqllogictest/src/util.rs
+++ b/datafusion/sqllogictest/src/util.rs
@@ -95,7 +95,7 @@ pub fn df_value_validator(
             warn!("[{i}] {}<eol>", normalized_actual[i]);
             warn!(
                 "[{i}] {}<eol>",
-                if normalized_expected.len() >= i {
+                if normalized_expected.len() > i {
                     &normalized_expected[i]
                 } else {
                     "No more results"


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

Reply via email to