From cac4651da07c0994754d32a1d2645e04067498c7 Mon Sep 17 00:00:00 2001 From: Gian Merlino Date: Tue, 26 Jan 2016 21:50:40 -0800 Subject: [PATCH 1/2] Fix spelling of 'dimReverseExtractionNamespace'. --- .../java/io/druid/query/extraction/NamespacedExtractor.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/extensions/namespace-lookup/src/main/java/io/druid/query/extraction/NamespacedExtractor.java b/extensions/namespace-lookup/src/main/java/io/druid/query/extraction/NamespacedExtractor.java index 1acfd1a155f..659adb3f8f6 100644 --- a/extensions/namespace-lookup/src/main/java/io/druid/query/extraction/NamespacedExtractor.java +++ b/extensions/namespace-lookup/src/main/java/io/druid/query/extraction/NamespacedExtractor.java @@ -49,7 +49,7 @@ public class NamespacedExtractor extends LookupExtractor public NamespacedExtractor( @NotNull @JacksonInject @Named("dimExtractionNamespace") final Function> namespaces, - @NotNull @JacksonInject @Named("reverseDimExtractionNamespace") + @NotNull @JacksonInject @Named("dimReverseExtractionNamespace") final Function>> reverseNamespaces, @NotNull @JsonProperty(value = "namespace", required = true) final String namespace From b1e6c017625d1102c900afadbc57657e63aa43ac Mon Sep 17 00:00:00 2001 From: Gian Merlino Date: Tue, 26 Jan 2016 23:02:53 -0800 Subject: [PATCH 2/2] Make LookupExtractor abstract methods public, they have to work across classloaders. --- .../java/io/druid/query/extraction/LookupExtractor.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/processing/src/main/java/io/druid/query/extraction/LookupExtractor.java b/processing/src/main/java/io/druid/query/extraction/LookupExtractor.java index 091f1f1cfd1..2de937df05e 100644 --- a/processing/src/main/java/io/druid/query/extraction/LookupExtractor.java +++ b/processing/src/main/java/io/druid/query/extraction/LookupExtractor.java @@ -44,7 +44,7 @@ public abstract class LookupExtractor * @return The lookup, or null key cannot have the lookup applied to it and should be treated as missing. */ @Nullable - abstract String apply(@NotNull String key); + public abstract String apply(@NotNull String key); /** * @param keys set of keys to apply lookup for each element @@ -54,7 +54,7 @@ public abstract class LookupExtractor * User can override this method if there is a better way to perform bulk lookup */ - Map applyAll(Iterable keys) + public Map applyAll(Iterable keys) { if (keys == null) { return Collections.emptyMap(); @@ -88,7 +88,7 @@ public abstract class LookupExtractor * User can override this method if there is a better way to perform bulk reverse lookup */ - Map> unapplyAll(Iterable values) + public Map> unapplyAll(Iterable values) { if (values == null) { return Collections.emptyMap(); @@ -107,5 +107,5 @@ public abstract class LookupExtractor */ @Nullable - abstract byte[] getCacheKey(); + public abstract byte[] getCacheKey(); }