Skip to content

Commit

Permalink
fix: auto compaction should be awaited too
Browse files Browse the repository at this point in the history
  • Loading branch information
py023 committed Dec 23, 2024
1 parent 5bc18f6 commit 47b35e7
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion regression-test/plugins/plugin_curl_requester.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -344,7 +344,9 @@ Suite.metaClass.trigger_and_wait_compaction = { String table_name, String compac
assert exit_code == 0: "trigger compaction failed, exit code: ${exit_code}, stdout: ${stdout}, stderr: ${stderr}"
def trigger_status = parseJson(stdout.trim())
if (trigger_status.status.toLowerCase() != "success") {
if (!auto_compaction_disabled) {
if (trigger_status.status.toLowerCase() == "already_exist") {
triggered_tablets.add(tablet) // compaction already in queue, treat it as successfully triggered
} else if (!auto_compaction_disabled) {
// ignore the error if auto compaction enabled
} else {
throw new Exception("trigger compaction failed, be host: ${be_host}, tablet id: ${tablet.TabletId}, status: ${trigger_status.status}")
Expand Down

0 comments on commit 47b35e7

Please sign in to comment.