From c40a013a1a95fd8ff985717e9502bea5535c8951 Mon Sep 17 00:00:00 2001 From: Gian Merlino Date: Thu, 2 Jun 2016 18:27:53 -0700 Subject: [PATCH] Improve NPE message in LookupDimensionSpec when lookup does not exist. The message used to be empty, which made things hard to debug. --- .../io/druid/query/dimension/LookupDimensionSpec.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/processing/src/main/java/io/druid/query/dimension/LookupDimensionSpec.java b/processing/src/main/java/io/druid/query/dimension/LookupDimensionSpec.java index fdbd3fa1c329..06d9eb91c8c6 100644 --- a/processing/src/main/java/io/druid/query/dimension/LookupDimensionSpec.java +++ b/processing/src/main/java/io/druid/query/dimension/LookupDimensionSpec.java @@ -26,10 +26,10 @@ import com.google.common.base.Strings; import com.metamx.common.StringUtils; import io.druid.query.extraction.ExtractionFn; +import io.druid.query.filter.DimFilterCacheHelper; import io.druid.query.lookup.LookupExtractionFn; import io.druid.query.lookup.LookupExtractor; import io.druid.query.lookup.LookupReferencesManager; -import io.druid.query.filter.DimFilterCacheHelper; import io.druid.segment.DimensionSelector; import javax.annotation.Nullable; @@ -129,10 +129,11 @@ public ExtractionFn getExtractionFn() final LookupExtractor lookupExtractor = Strings.isNullOrEmpty(name) ? this.lookup : Preconditions.checkNotNull( - this.lookupReferencesManager.get(name).get(), - "can not find lookup with name [%s]", + lookupReferencesManager.get(name), + "Lookup [%s] not found", name - ); + ).get(); + return new LookupExtractionFn( lookupExtractor, retainMissingValue, @@ -140,7 +141,6 @@ public ExtractionFn getExtractionFn() lookupExtractor.isOneToOne(), optimize ); - } @Override