Skip to content

branch-4.0: [fix](test) deflake test_curd_wlg and adapt test_single_replica_compaction_with_format_v2 to V3 default #64666

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
morningman merged 1 commit into from
Jun 22, 2026
Merged

branch-4.0: [fix](test) deflake test_curd_wlg and adapt test_single_replica_compaction_with_format_v2 to V3 default #64666

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion ...es/inverted_index_p0/index_format_v2/test_single_replica_compaction_with_format_v2.groovy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,12 @@ suite("test_single_replica_compaction_with_format_v2", "inverted_index_format_v2
backend_id = tablet.BackendId
String ip = backendId_to_backendIP.get(backend_id)
String port = backendId_to_backendHttpPort.get(backend_id)
check_nested_index_file(ip, port, tablet_id, 2, 3, "V2")
// The default inverted_index_storage_format changed from V2 to V3 (#57252), and the
// post-compaction rowset count varies per replica, so derive it dynamically and check
// against V3 (mirrors test_cumulative_compaction_with_format_v2 / test_mow_table_with_format_v2).
def (tablet_status_code, tablet_status_out, tablet_status_err) = be_show_tablet_status(ip, port, tablet_id)
int activeRowsetCount = parseJson(tablet_status_out.trim()).rowsets.size()
check_nested_index_file(ip, port, tablet_id, activeRowsetCount, 3, "V3")
}

int segmentsCount = 0
Expand Down
5 changes: 5 additions & 0 deletions regression-test/suites/workload_manager_p0/test_curd_wlg.groovy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -417,10 +417,15 @@ suite("test_crud_wlg") {
// test bypass
sql "create workload group if not exists bypass_group $forComputeGroupStr properties ( 'max_concurrency'='0','max_queue_size'='0','queue_timeout'='0');"
sql "set workload_group=bypass_group;"
// Disable the FE SQL cache here: `select count(1) from ${table_name}` is run repeatedly in this
// suite, so it can be served from the result cache and skip the Coordinator/query-queue path
// entirely, which makes this "queue is full" assertion flaky (the rejection never fires).
sql "set enable_sql_cache=false;"
test {
sql "select count(1) from ${table_name};"
exception "query waiting queue is full"
}
sql "set enable_sql_cache=true;"

sql "set bypass_workload_group = true;"
sql "select count(1) from information_schema.active_queries;"
Expand Down
Loading