From c5e2a8818791ac87dafb5bc5fbbbf1de3684cee1 Mon Sep 17 00:00:00 2001 From: Ian Brandt Date: Tue, 30 Oct 2012 23:59:31 -0700 Subject: [PATCH] Resolved dead code warning by null checking the correct variable according to the associated log statement. --- server/src/main/java/com/metamx/druid/master/DruidMaster.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/src/main/java/com/metamx/druid/master/DruidMaster.java b/server/src/main/java/com/metamx/druid/master/DruidMaster.java index f42021606bf..0a90e7f99da 100644 --- a/server/src/main/java/com/metamx/druid/master/DruidMaster.java +++ b/server/src/main/java/com/metamx/druid/master/DruidMaster.java @@ -359,7 +359,7 @@ public class DruidMaster DataSegment newDataSegment = sizeAdjuster.updateDescriptor(dataSegment); - if (dataSegment == null) { + if (newDataSegment == null) { log.warn("newDataSegment was null with old dataSegment[%s]. Skipping.", dataSegment); continue; }