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

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


The following commit(s) were added to refs/heads/branch-4.1 by this push:
     new 81e7edab16a [fix](regression) Fix test_query_stmt size assertion 
(#65256)
81e7edab16a is described below

commit 81e7edab16ab6c41c7bead8e3758f162090db03c
Author: shuke <[email protected]>
AuthorDate: Tue Jul 7 13:56:30 2026 +0800

    [fix](regression) Fix test_query_stmt size assertion (#65256)
    
    ## Summary
    
    Pick master PR #62455 to branch-4.1 for
    `http_rest_api.post.test_query_stmt`.
    
    The branch-4.1 case still uses Groovy property access:
    
    ```groovy
    obj.data.data.size
    ```
    
    Under the current regression runtime this can be interpreted as GPath
    property access over the nested list and fail with:
    
    ```text
    groovy.lang.MissingPropertyException: No such property: size for class: 
java.lang.Integer
    ```
    
    Use the explicit method call instead:
    
    ```groovy
    obj.data.data.size()
    ```
    
    ## Testing
    
    - [x] Cherry-picked master commit `50cbb9f82a1` (`apache/doris#62455`)
    onto branch-4.1.
    - [x] Verified the branch diff only changes
    `regression-test/suites/http_rest_api/post/test_query_stmt.groovy`.
    - [ ] `run buildall`
    
    Co-authored-by: Claude Sonnet 4.6 <[email protected]>
---
 regression-test/suites/http_rest_api/post/test_query_stmt.groovy | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/regression-test/suites/http_rest_api/post/test_query_stmt.groovy 
b/regression-test/suites/http_rest_api/post/test_query_stmt.groovy
index 3001da2b15d..eba5ecc11f5 100644
--- a/regression-test/suites/http_rest_api/post/test_query_stmt.groovy
+++ b/regression-test/suites/http_rest_api/post/test_query_stmt.groovy
@@ -152,7 +152,7 @@ suite("test_query_stmt") {
     assertEquals(obj.msg, SUCCESS_MSG)
     assertEquals(obj.code, SUCCESS_CODE)
     // we can only check the number is correctly
-    assertEquals(obj.data.data.size, 3)
+    assertEquals(obj.data.data.size(), 3)
 
     url = "/api/query_schema/default_cluster/" + context.config.defaultDb
     def stmt5 = " select * from ${tableName}"


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

Reply via email to