rename

Author: martijnvg

server/src/main/java/org/elasticsearch/action/index/IndexRequest.java

@@ -776,7 +776,7 @@ public Index getConcreteWriteIndex(IndexAbstraction ia, Metadata metadata) {
                     );
             }
 
-            Index result = dataStream.selectWriteIndex(timestamp, metadata);
+            Index result = dataStream.selectTimeSeriesWriteIndex(timestamp, metadata);
             if (result == null) {
                 String timestampAsString = DateFieldMapper.DEFAULT_DATE_TIME_FORMATTER.format(timestamp);
                 throw new IllegalArgumentException("no index available for a document with an @timestamp of [" + timestampAsString + "]");

server/src/main/java/org/elasticsearch/cluster/metadata/DataStream.java

@@ -175,7 +175,13 @@ public Index getWriteIndex() {
         return indices.get(indices.size() - 1);
     }
 
-    public Index selectWriteIndex(Instant timestamp, Metadata metadata) {
+    /**
+     * @param timestamp The timestamp used to select a backing index based on its start and end time.
+     * @param metadata  The metadata that is used to fetch the start and end times for backing indices of this data stream.
+     * @return a backing index with a start time that is greater or equal to the provided timestamp and
+     *         an end time that is less than the provided timestamp. Otherwise <code>null</code> is returned.
+     */
+    public Index selectTimeSeriesWriteIndex(Instant timestamp, Metadata metadata) {
         for (int i = indices.size() - 1; i >= 0; i--) {
             Index index = indices.get(i);
             IndexMetadata im = metadata.index(index);

server/src/test/java/org/elasticsearch/cluster/metadata/DataStreamTests.java

@@ -484,7 +484,7 @@ public void testSnapshotWithAllBackingIndicesRemoved() {
         );
     }
 
-    public void testSelectWriteIndex() {
+    public void testSelectTimeSeriesWriteIndex() {
         Instant currentTime = Instant.now();
 
         Instant start1 = currentTime.minus(6, ChronoUnit.HOURS);
@@ -499,19 +499,19 @@ public void testSelectWriteIndex() {
         );
 
         DataStream dataStream = clusterState.getMetadata().dataStreams().get(dataStreamName);
-        Index result = dataStream.selectWriteIndex(currentTime, clusterState.getMetadata());
+        Index result = dataStream.selectTimeSeriesWriteIndex(currentTime, clusterState.getMetadata());
         assertThat(result, equalTo(dataStream.getIndices().get(1)));
         assertThat(result.getName(), equalTo(DataStream.getDefaultBackingIndexName(dataStreamName, 2, start2.toEpochMilli())));
 
-        result = dataStream.selectWriteIndex(currentTime.minus(2, ChronoUnit.HOURS), clusterState.getMetadata());
+        result = dataStream.selectTimeSeriesWriteIndex(currentTime.minus(2, ChronoUnit.HOURS), clusterState.getMetadata());
         assertThat(result, equalTo(dataStream.getIndices().get(1)));
         assertThat(result.getName(), equalTo(DataStream.getDefaultBackingIndexName(dataStreamName, 2, start2.toEpochMilli())));
 
-        result = dataStream.selectWriteIndex(currentTime.minus(3, ChronoUnit.HOURS), clusterState.getMetadata());
+        result = dataStream.selectTimeSeriesWriteIndex(currentTime.minus(3, ChronoUnit.HOURS), clusterState.getMetadata());
         assertThat(result, equalTo(dataStream.getIndices().get(0)));
         assertThat(result.getName(), equalTo(DataStream.getDefaultBackingIndexName(dataStreamName, 1, start1.toEpochMilli())));
 
-        result = dataStream.selectWriteIndex(currentTime.minus(6, ChronoUnit.HOURS), clusterState.getMetadata());
+        result = dataStream.selectTimeSeriesWriteIndex(currentTime.minus(6, ChronoUnit.HOURS), clusterState.getMetadata());
         assertThat(result, equalTo(dataStream.getIndices().get(0)));
         assertThat(result.getName(), equalTo(DataStream.getDefaultBackingIndexName(dataStreamName, 1, start1.toEpochMilli())));
     }