[es] Remove unused indexCache (jaegertracing#5408)

## Which problem is this PR solving?
- The cache was initialized with incorrect parameter, and when I looked
around it wasn't even used anywhere

## Description of the changes
- delete indexCache and indexCacheTTL argument

## How was this change tested?
```
$ go test ./plugin/storage/es/spanstore
ok      github.com/jaegertracing/jaeger/plugin/storage/es/spanstore     0.471s
```

Signed-off-by: Yuri Shkuro <[email protected]>

Author: yurishkuro

plugin/storage/es/spanstore/writer.go

@@ -46,10 +46,10 @@ type serviceWriter func(string, *dbmodel.Span)
 
 // SpanWriter is a wrapper around elastic.Client
 type SpanWriter struct {
-	client           func() es.Client
-	logger           *zap.Logger
-	writerMetrics    spanWriterMetrics // TODO: build functions to wrap around each Do fn
-	indexCache       cache.Cache
+	client        func() es.Client
+	logger        *zap.Logger
+	writerMetrics spanWriterMetrics // TODO: build functions to wrap around each Do fn
+	// indexCache       cache.Cache
 	serviceWriter    serviceWriter
 	spanConverter    dbmodel.FromDomain
 	spanServiceIndex spanAndServiceIndexFn
@@ -69,7 +69,6 @@ type SpanWriterParams struct {
 	Archive                bool
 	UseReadWriteAliases    bool
 	ServiceCacheTTL        time.Duration
-	IndexCacheTTL          time.Duration
 }
 
 // NewSpanWriter creates a new SpanWriter for use
@@ -79,25 +78,14 @@ func NewSpanWriter(p SpanWriterParams) *SpanWriter {
 		serviceCacheTTL = serviceCacheTTLDefault
 	}
 
-	indexCacheTTL := p.IndexCacheTTL
-	if p.ServiceCacheTTL == 0 {
-		indexCacheTTL = indexCacheTTLDefault
-	}
-
 	serviceOperationStorage := NewServiceOperationStorage(p.Client, p.Logger, serviceCacheTTL)
 	return &SpanWriter{
 		client: p.Client,
 		logger: p.Logger,
 		writerMetrics: spanWriterMetrics{
 			indexCreate: storageMetrics.NewWriteMetrics(p.MetricsFactory, "index_create"),
 		},
-		serviceWriter: serviceOperationStorage.Write,
-		indexCache: cache.NewLRUWithOptions(
-			5,
-			&cache.Options{
-				TTL: indexCacheTTL,
-			},
-		),
+		serviceWriter:    serviceOperationStorage.Write,
 		spanConverter:    dbmodel.NewFromDomain(p.AllTagsAsFields, p.TagKeysAsFields, p.TagDotReplacement),
 		spanServiceIndex: getSpanAndServiceIndexFn(p.Archive, p.UseReadWriteAliases, p.IndexPrefix, p.SpanIndexDateLayout, p.ServiceIndexDateLayout),
 	}

plugin/storage/es/spanstore/writer_test.go

@@ -424,19 +424,16 @@ func TestSpanWriterParamsTTL(t *testing.T) {
 	logger, _ := testutils.NewLogger()
 	metricsFactory := metricstest.NewFactory(0)
 	testCases := []struct {
-		indexTTL         time.Duration
 		serviceTTL       time.Duration
 		name             string
 		expectedAddCalls int
 	}{
 		{
-			indexTTL:         0,
 			serviceTTL:       0,
 			name:             "uses defaults",
 			expectedAddCalls: 1,
 		},
 		{
-			indexTTL:         1 * time.Nanosecond,
 			serviceTTL:       1 * time.Nanosecond,
 			name:             "uses provided values",
 			expectedAddCalls: 3,
@@ -451,7 +448,6 @@ func TestSpanWriterParamsTTL(t *testing.T) {
 				Logger:          logger,
 				MetricsFactory:  metricsFactory,
 				ServiceCacheTTL: test.serviceTTL,
-				IndexCacheTTL:   test.indexTTL,
 			}
 			w := NewSpanWriter(params)