morningman opened a new pull request, #63644:
URL: https://github.com/apache/doris/pull/63644

   ## Summary
   
   Replace `res.size` / `res[i].size` property accesses with `res.size()` / 
`res[i].size()` method calls in 
`regression-test/plugins/plugins_create_table_nested_type.groovy`.
   
   On the Groovy runtime used by the regression framework, 
`java.util.ArrayList` does not expose `size` as a JavaBean property (there is 
no `getSize()`), so the property access falls through to the `getAt(Iterable, 
...)` path, which iterates the outer list and tries to read `.size` from each 
inner `Integer` element, throwing:
   
   ```
   groovy.lang.MissingPropertyException: Exception evaluating property 'size' 
for
   java.util.ArrayList, Reason: groovy.lang.MissingPropertyException:
   No such property: size for class: java.lang.Integer
   ```
   
   This makes the test 
`datatype_p0.nested_types.create_table.create_table_with_nested_type` fail in 
~15 ms before any DDL is sent to FE. The test is currently muted on TeamCity 
(under investigation); this fix should unblock unmuting.
   
   ```diff
   -    for (int i = 0; i < res.size; i++) {
   +    for (int i = 0; i < res.size(); i++) {
            def date_type_str = ""
   -        for (int j = 0; j < res[i].size; j++) {
   +        for (int j = 0; j < res[i].size(); j++) {
                date_type_str += res[i][j] + " "
            }
   ```
   
   Note: this file is byte-identical with `apache/master` and the same bug 
exists upstream. A corresponding fix on master is recommended as a follow-up.
   
   ## Test plan
   
   - [ ] Once merged, unmute `datatype_p0.create_table_with_nested_type` and 
confirm the suite at least progresses past the closure entry point 
(Groovy-layer error is resolved).
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to