Merge pull request apache#2343 from gianm/fix-lookups

Fix a couple of problems with lookups
SnowCicala · Jan 27, 2016 · abf2f45 · abf2f45
2 parents cac6753 + b1e6c01
commit abf2f45
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
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
@@ -49,7 +49,7 @@ public class NamespacedExtractor extends LookupExtractor
   public NamespacedExtractor(
       @NotNull @JacksonInject @Named("dimExtractionNamespace")
       final Function<String, Function<String, String>> namespaces,
-      @NotNull @JacksonInject @Named("reverseDimExtractionNamespace")
+      @NotNull @JacksonInject @Named("dimReverseExtractionNamespace")
       final Function<String, Function<String, List<String>>> reverseNamespaces,
       @NotNull @JsonProperty(value = "namespace", required = true)
       final String namespace

diff --git 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<String, String> applyAll(Iterable<String> keys)
+  public Map<String, String> applyAll(Iterable<String> keys)
   {
     if (keys == null) {
       return Collections.emptyMap();
@@ -88,7 +88,7 @@ Map<String, String> applyAll(Iterable<String> keys)
    * User can override this method if there is a better way to perform bulk reverse lookup
    */
 
-  Map<String, List<String>> unapplyAll(Iterable<String> values)
+  public Map<String, List<String>> unapplyAll(Iterable<String> values)
   {
     if (values == null) {
       return Collections.emptyMap();
@@ -107,5 +107,5 @@ Map<String, List<String>> unapplyAll(Iterable<String> values)
    */
 
   @Nullable
-  abstract byte[] getCacheKey();
+  public abstract byte[] getCacheKey();
 }