istio / old_mixer_repo

Deprecated home of Istio's Mixer and its adapters, now in istio/istio's mixer dir
https://github.com/istio/istio/tree/master/mixer
Apache License 2.0
67 stars 93 forks source link

Add an expression cache #1487

Closed geeknoid closed 6 years ago

geeknoid commented 7 years ago

We need to assess whether introducing a cache in front of the expression evaluation logic is desirable or not. If it is, we should add such a cache.

We need to determine whether the overhead of maintaining a cache is worth it. Expressions usually tend to be pretty short, so perhaps caching wouldn't bring us much.

ozevren commented 6 years ago

Resolving this as a duplicate of :https://github.com/istio/istio/issues/3572 While 3572 is newer, it is also in the new depot.