Remove cache.OperatorCacheProvider interface. (#2680)

benluddy · web-flow · commit ddf92b1903cf · 2022-03-02T11:22:27.000-05:00
This interface is unused, so it can be removed as a step toward a
stable and supported cache package API.

Signed-off-by: Ben Luddy &lt;bluddy@redhat.com&gt;
diff --git a/pkg/controller/registry/resolver/cache/cache.go b/pkg/controller/registry/resolver/cache/cache.go
@@ -90,8 +90,6 @@ type Cache struct {
 	m                      sync.RWMutex
 }
 
-var _ OperatorCacheProvider = &Cache{}
-
 type Option func(*Cache)
 
 func WithLogger(logger logrus.StdLogger) Option {
diff --git a/pkg/controller/registry/resolver/resolver.go b/pkg/controller/registry/resolver/resolver.go
@@ -29,7 +29,7 @@ type constraintProvider interface {
 }
 
 type Resolver struct {
-	cache                     cache.OperatorCacheProvider
+	cache                     *cache.Cache
 	log                       logrus.FieldLogger
 	pc                        *predicateConverter
 	systemConstraintsProvider constraintProvider

Original file line number	Diff line number	Diff line change
`@@ -90,8 +90,6 @@ type Cache struct {`
`90`	`90`	`m sync.RWMutex`
`91`	`91`	`}`
`92`	`92`
`93`		`-var _ OperatorCacheProvider = &Cache{}`
`94`		`-`
`95`	`93`	`type Option func(*Cache)`
`96`	`94`
`97`	`95`	`func WithLogger(logger logrus.StdLogger) Option {`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ type constraintProvider interface {`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`type Resolver struct {`
`32`		`- cache cache.OperatorCacheProvider`
	`32`	`+ cache *cache.Cache`
`33`	`33`	`log logrus.FieldLogger`
`34`	`34`	`pc *predicateConverter`
`35`	`35`	`systemConstraintsProvider constraintProvider`