This is an automated email from the ASF dual-hosted git repository.
Jackie-Jiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new a7a9761605a Allow zero system activities CPU time in query killing
tests (#20159)
a7a9761605a is described below
commit a7a9761605a50f76ef4005f7148da57a7b3ddfff
Author: Xiaotian (Jackie) Jiang <[email protected]>
AuthorDate: Thu Nov 5 11:20:27 2026 -0800
Allow zero system activities CPU time in query killing tests (#20159)
---
.../pinot/integration/tests/QueryKillingIntegrationTest.java | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git
a/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/QueryKillingIntegrationTest.java
b/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/QueryKillingIntegrationTest.java
index 45a1c4577f1..311e0df65ac 100644
---
a/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/QueryKillingIntegrationTest.java
+++
b/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/QueryKillingIntegrationTest.java
@@ -216,7 +216,8 @@ public class QueryKillingIntegrationTest extends
BaseClusterIntegrationTest {
long offlineResponseSerializationCpuTimeNs =
queryResponse.get("offlineResponseSerializationCpuTimeNs").asLong();
long offlineTotalCpuTimeNs =
queryResponse.get("offlineTotalCpuTimeNs").asLong();
assertTrue(offlineThreadCpuTimeNs > 0);
- assertTrue(offlineSystemActivitiesCpuTimeNs > 0);
+ // System activities time is an estimate of unaccounted wall time and can
be clamped to zero.
+ assertTrue(offlineSystemActivitiesCpuTimeNs >= 0);
assertTrue(offlineResponseSerializationCpuTimeNs > 0);
assertEquals(offlineThreadCpuTimeNs + offlineSystemActivitiesCpuTimeNs +
offlineResponseSerializationCpuTimeNs,
offlineTotalCpuTimeNs);
@@ -366,8 +367,9 @@ public class QueryKillingIntegrationTest extends
BaseClusterIntegrationTest {
long offlineTotalCpuTimeNs =
queryResponse.get("offlineTotalCpuTimeNs").asLong();
assertTrue(offlineThreadCpuTimeNs > 0,
"offlineThreadCpuTimeNs should be > 0 after killing tests");
- assertTrue(offlineSystemActivitiesCpuTimeNs > 0,
- "offlineSystemActivitiesCpuTimeNs should be > 0 after killing tests");
+ // System activities time is an estimate of unaccounted wall time and can
be clamped to zero.
+ assertTrue(offlineSystemActivitiesCpuTimeNs >= 0,
+ "offlineSystemActivitiesCpuTimeNs should be >= 0 after killing tests");
assertTrue(offlineResponseSerializationCpuTimeNs > 0,
"offlineResponseSerializationCpuTimeNs should be > 0 after killing
tests");
assertEquals(offlineThreadCpuTimeNs + offlineSystemActivitiesCpuTimeNs +
offlineResponseSerializationCpuTimeNs,
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
