[fix] (compaction) fix time series compaction policy (#39170)

csun5285 · web-flow · commit 191371df057b · 2024-08-12T12:10:03.000+08:00
## Proposed changes Fix the issue introduced by #38220 ### BUG The permits requested for compaction cannot be 0, 0 indicates failure.
diff --git a/be/src/olap/tablet.cpp b/be/src/olap/tablet.cpp
@@ -1723,7 +1723,8 @@ Status Tablet::prepare_compaction_and_calculate_permits(
 
     // Time series policy does not rely on permits, it uses goal size to control memory
     if (tablet->tablet_meta()->compaction_policy() == CUMULATIVE_TIME_SERIES_POLICY) {
-        permits = 0;
+        // permits = 0 means that prepare_compaction failed
+        permits = 1;
     } else {
         permits = compaction->get_compaction_permits();
     }

Original file line number	Diff line number	Diff line change
`@@ -1723,7 +1723,8 @@ Status Tablet::prepare_compaction_and_calculate_permits(`
`1723`	`1723`
`1724`	`1724`	`// Time series policy does not rely on permits, it uses goal size to control memory`
`1725`	`1725`	`if (tablet->tablet_meta()->compaction_policy() == CUMULATIVE_TIME_SERIES_POLICY) {`
`1726`		`- permits = 0;`
	`1726`	`+ // permits = 0 means that prepare_compaction failed`
	`1727`	`+ permits = 1;`
`1727`	`1728`	`} else {`
`1728`	`1729`	`permits = compaction->get_compaction_permits();`
`1729`	`1730`	`}`