Remove old TODO's that are handled, irrelevant now, or will be handled in other PR's currently in flight

Author: jbaiera

server/src/main/java/org/elasticsearch/action/search/TransportSearchAction.java

@@ -1595,7 +1595,6 @@ public String[] indices() {
 
                     @Override
                     public IndicesOptions indicesOptions() {
-                        // PRTODO: What should this allow?
                         return IndicesOptions.strictSingleIndexNoExpandForbidClosed();
                     }
                 });

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

@@ -206,8 +206,6 @@ public String[] concreteIndexNames(ClusterState state, IndicesOptions options, I
     }
 
     public List<String> dataStreamNames(ClusterState state, IndicesOptions options, String... indexExpressions) {
-        // PRTODO: We should assert that callers of this have selectors disallowed, and direct them to the other method if they do.
-        // SnapshotService CreateSnapshotRequest
         return dataStreams(state, options, indexExpressions).stream().map(ResolvedExpression::resource).distinct().toList();
     }
 

server/src/main/java/org/elasticsearch/snapshots/SnapshotsService.java

@@ -787,8 +787,6 @@ private static Metadata metadataForSnapshot(SnapshotsInProgress.Entry snapshot,
                     : "Data stream [" + dataStreamName + "] was deleted during a snapshot but snapshot was not partial.";
             } else {
                 boolean missingIndex = false;
-                // PRTODO: Does this need to be updated? If we take a snapshot of `logs-*::data` and then recover it after deleting the
-                // stream, will the data stream have a bunch of unreferenced index names in its failure store list?
                 for (Index index : dataStream.getIndices()) {
                     final String indexName = index.getName();
                     if (builder.get(indexName) == null || indicesInSnapshot.contains(indexName) == false) {