This is an automated email from the ASF dual-hosted git repository.
hello-stephen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 5f0d964d79d [fix](regression-test) fix two muted test cases (Groovy
property access + flaky SHOW PROCESSLIST) (#63645)
5f0d964d79d is described below
commit 5f0d964d79df442c0695bb7a1e0d9b309692897e
Author: Mingyu Chen (Rayner) <[email protected]>
AuthorDate: Mon Jun 22 14:30:25 2026 +0800
[fix](regression-test) fix two muted test cases (Groovy property access +
flaky SHOW PROCESSLIST) (#63645)
## Summary
Fix two regression-test cases that are currently muted on branch-4.0
(and have the same bugs on master, just not yet observed/muted). Neither
requires any FE/BE changes — both are Groovy / test-harness issues.
The companion back-port to `branch-4.0` is apache/doris#63644.
### 1. `datatype_p0.create_table_with_nested_type`
`regression-test/plugins/plugins_create_table_nested_type.groovy:136,138`
```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] + " "
}
```
On the Groovy runtime used by the regression framework,
`java.util.ArrayList` does not expose `size` as a JavaBean property, so
`res.size` falls through GPath into `getAt(Iterable, ...)`, iterates the
outer list, and tries to read `.size` from each inner `Integer` —
throwing `MissingPropertyException: No such property: size for class:
java.lang.Integer`. The test fails in ~15 ms before any DDL is sent to
FE.
Same root cause as #62454 / #62455.
### 2. `auth_call.test_show_charset_auth.test_show_no_auth`
`regression-test/suites/auth_call/test_show_charset_auth.groovy:68`
```diff
- assertTrue(res1.size() == 1)
+ def ownSessions = res1.findAll { it[2] == user }
+ assertTrue(ownSessions.size() >= 1)
```
After granting `grant_priv`, `SHOW PROCESSLIST` returns *all* sessions
to any caller that satisfies `PrivPredicate.ADMIN` (see
`ConnectPoolMgr.listConnection`), so in the parallel pipeline this user
sees 20+ concurrent sessions from other suites. The `== 1` assertion is
structurally unsafe — replace it with a per-user filter that matches the
same pattern already used at line 49 of the file. Column index 2 is the
User column (`ConnectContext.ThreadInfo.toRow`).
## Test plan
- [ ] After merge, unmute the two test cases on TeamCity (branch-4.0):
- `datatype_p0.nested_types.create_table.create_table_with_nested_type`
- `auth_call.test_show_charset_auth.test_show_no_auth`
- [ ] Confirm each test passes (or surfaces a different, real failure)
on the next P0 run.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
---------
Co-authored-by: Claude Opus 4.7 (1M context) <[email protected]>
---
regression-test/suites/auth_call/test_show_charset_auth.groovy | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/regression-test/suites/auth_call/test_show_charset_auth.groovy
b/regression-test/suites/auth_call/test_show_charset_auth.groovy
index d1b69a51992..134e53141cb 100644
--- a/regression-test/suites/auth_call/test_show_charset_auth.groovy
+++ b/regression-test/suites/auth_call/test_show_charset_auth.groovy
@@ -65,7 +65,8 @@ suite("test_show_no_auth","p0,auth_call") {
def res1 = sql """SHOW PROCESSLIST"""
logger.info("res1: " + res1)
- assertTrue(res1.size() == 1)
+ def ownSessions = res1.findAll { it[2] == user }
+ assertTrue(ownSessions.size() >= 1)
}
sql """revoke grant_priv on *.*.* from ${user}"""
sql """grant admin_priv on *.*.* to ${user}"""
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
