Skip to content

Commit

Permalink
[Fix](regression) Fix flaky test `fault_injection_p0/cloud/test_cloud…
Browse files Browse the repository at this point in the history
…_mow_stale_resp_load_load_conflict` (#43448)

Problem Summary: `ADMIN SET FRONTEND CONFIG()` will only update the
config of the connected FE, which may lead to unexpected result for some
cases if the connected FE is not master FE. should use `ADMIN SET
FRONTEND CONFIG()` instead.
  • Loading branch information
bobhan1 authored and Your Name committed Nov 13, 2024
1 parent 27e0051 commit 8a6f264
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -1622,7 +1622,7 @@ class Suite implements GroovyInterceptable {
}

void setFeConfig(String key, Object value) {
sql "ADMIN SET FRONTEND CONFIG ('${key}' = '${value}')"
sql "ADMIN SET ALL FRONTEND CONFIG ('${key}' = '${value}')"
}

void setFeConfigTemporary(Map<String, Object> tempConfig, Closure actionSupplier) {
Expand Down

0 comments on commit 8a6f264

Please sign in to comment.