This is an automated email from the ASF dual-hosted git repository.
kfaraz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/master by this push:
new 2b02e7a28ae test: Silence unrelated NPE alerts in DruidCoordinatorTest
(#19953)
2b02e7a28ae is described below
commit 2b02e7a28aebc0f7ce6d211d2ca1ebbbbced4139
Author: Abhishek Radhakrishnan <[email protected]>
AuthorDate: Mon Aug 10 20:52:19 2026 -0700
test: Silence unrelated NPE alerts in DruidCoordinatorTest (#19953)
The shared test MetadataManager intentionally has null supervisor/audit/
indexer managers. With metadata-store cleanup enabled (the default), the
Kill* duties dereference those nulls and emit unrelated NPE alerts. Disable
metadata cleanup in the shared coordinator config so every test that starts
a real coordinator stops emitting the noise.
---
.../server/coordinator/DruidCoordinatorTest.java | 20 +++++++++++++++++++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff --git
a/server/src/test/java/org/apache/druid/server/coordinator/DruidCoordinatorTest.java
b/server/src/test/java/org/apache/druid/server/coordinator/DruidCoordinatorTest.java
index 952bce46a6d..20013ef3775 100644
---
a/server/src/test/java/org/apache/druid/server/coordinator/DruidCoordinatorTest.java
+++
b/server/src/test/java/org/apache/druid/server/coordinator/DruidCoordinatorTest.java
@@ -52,6 +52,7 @@ import
org.apache.druid.server.coordinator.config.CoordinatorKillConfigs;
import org.apache.druid.server.coordinator.config.CoordinatorPeriodConfig;
import org.apache.druid.server.coordinator.config.CoordinatorRunConfig;
import org.apache.druid.server.coordinator.config.DruidCoordinatorConfig;
+import org.apache.druid.server.coordinator.config.MetadataCleanupConfig;
import org.apache.druid.server.coordinator.duty.CompactSegments;
import org.apache.druid.server.coordinator.duty.CoordinatorCustomDuty;
import org.apache.druid.server.coordinator.duty.CoordinatorCustomDutyGroup;
@@ -135,10 +136,27 @@ public class DruidCoordinatorTest
).andReturn(new
AtomicReference<>(DruidCompactionConfig.empty())).anyTimes();
EasyMock.replay(configManager);
statusTracker = new CompactionStatusTracker();
+
+ final MetadataCleanupConfig cleanupDisabled = new
MetadataCleanupConfig(false, null, null);
druidCoordinatorConfig = new DruidCoordinatorConfig(
new CoordinatorRunConfig(new Duration(COORDINATOR_START_DELAY), new
Duration(COORDINATOR_PERIOD)),
new CoordinatorPeriodConfig(null, null),
- CoordinatorKillConfigs.DEFAULT,
+ new CoordinatorKillConfigs(
+ cleanupDisabled,
+ cleanupDisabled,
+ cleanupDisabled,
+ cleanupDisabled,
+ cleanupDisabled,
+ cleanupDisabled,
+ cleanupDisabled,
+ false,
+ null,
+ null,
+ null,
+ null,
+ null,
+ null
+ ),
new CostBalancerStrategyFactory(),
null
);
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
