Data tiers don't support aggregating and this failed listing supporte…

…d aggregations
elastic · Feb 18, 2023 · 8bf158e · 8bf158e
1 parent 24c5130
commit 8bf158e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/server/src/main/java/org/elasticsearch/index/mapper/ConstantFieldType.java b/server/src/main/java/org/elasticsearch/index/mapper/ConstantFieldType.java
@@ -35,11 +35,6 @@ public ConstantFieldType(String name, Map<String, String> meta) {
         assert isSearchable();
     }
 
-    @Override
-    public final boolean isAggregatable() {
-        return true;
-    }
-
     /**
      * Return whether the constant value of this field matches the provided {@code pattern}
      * as documented in {@link Regex#simpleMatch}.

diff --git a/...n/java/org/elasticsearch/xpack/cluster/routing/allocation/mapper/DataTierFieldMapper.java b/...n/java/org/elasticsearch/xpack/cluster/routing/allocation/mapper/DataTierFieldMapper.java
@@ -50,6 +50,12 @@ public String familyTypeName() {
             return KeywordFieldMapper.CONTENT_TYPE;
         }
 
+        @Override
+        public boolean isAggregatable() {
+            // Doesn't support field data (yet) and therefor attempting to aggregate on field of this field type fail
+            return false;
+        }
+
         @Override
         protected boolean matches(String pattern, boolean caseInsensitive, SearchExecutionContext context) {
             if (caseInsensitive) {