From 4bfd65a37506f5f4d5073ee14441b20fe97ef539 Mon Sep 17 00:00:00 2001
From: Julie Tibshirani <julie.tibshirani@elastic.co>
Date: Fri, 24 Apr 2020 17:03:45 -0700
Subject: [PATCH] Remove TODO around aggregating on _index.

The _index field can in fact be used in aggregations.
---
 .../java/org/elasticsearch/index/mapper/IndexFieldMapper.java    | 1 -
 1 file changed, 1 deletion(-)

diff --git a/server/src/main/java/org/elasticsearch/index/mapper/IndexFieldMapper.java b/server/src/main/java/org/elasticsearch/index/mapper/IndexFieldMapper.java
index 018a5df4ea2..5de733d4ea5 100644
--- a/server/src/main/java/org/elasticsearch/index/mapper/IndexFieldMapper.java
+++ b/server/src/main/java/org/elasticsearch/index/mapper/IndexFieldMapper.java
@@ -115,7 +115,6 @@ public class IndexFieldMapper extends MetadataFieldMapper {
 
         @Override
         public ValuesSourceType getValuesSourceType() {
-            // TODO: Should Index fields be aggregatable?  What even is an IndexField?
             return CoreValuesSourceType.BYTES;
         }
     }