return null instead of default

apache · Oct 20, 2023 · 247931b · 247931b
1 parent 11fd787
commit 247931b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/...n/java/org/apache/druid/query/rowsandcols/semantic/DefaultColumnSelectorFactoryMaker.java b/...n/java/org/apache/druid/query/rowsandcols/semantic/DefaultColumnSelectorFactoryMaker.java
@@ -162,7 +162,7 @@ public ColumnCapabilities getColumnCapabilities(String column)
     {
       return withColumnAccessor(column, columnAccessor -> {
         if (columnAccessor == null) {
-          return ColumnCapabilitiesImpl.createDefault();
+          return null;
         } else {
           return new ColumnCapabilitiesImpl()
               .setType(columnAccessor.getType())

diff --git a/...test/java/org/apache/druid/query/rowsandcols/semantic/CombinedSemanticInterfacesTest.java b/...test/java/org/apache/druid/query/rowsandcols/semantic/CombinedSemanticInterfacesTest.java
@@ -44,7 +44,7 @@
 import java.util.function.Function;
 
 import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNull;
 
 /**
  * Place where tests can live that are testing the interactions of multiple semantic interfaces
@@ -81,7 +81,7 @@ public void testColumnSelectorFactoryGetColumnCapabilitiesNonExistentColumn()
     AtomicInteger currRow = new AtomicInteger();
     ColumnSelectorFactory csfm = ColumnSelectorFactoryMaker.fromRAC(rac).make(currRow);
 
-    assertNotNull(csfm.getColumnCapabilities("nonexistent"));
+    assertNull(csfm.getColumnCapabilities("nonexistent"));
   }
 
   /**