Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Setting custom cache options when using MultiNamespacedCacheBuilder? #2273

Closed
tsaarni opened this issue Apr 12, 2023 · 2 comments
Closed

Setting custom cache options when using MultiNamespacedCacheBuilder? #2273

tsaarni opened this issue Apr 12, 2023 · 2 comments

Comments

@tsaarni
Copy link

tsaarni commented Apr 12, 2023

There are two different forms of builders:

  • cache.BuilderWithOptions() allows setting custom cache.Options like TransformByObject
  • cache.MultiNamespacedCacheBuilder() allows namespace-limited watches

Is it possible to create NewCacheFunc that combines both of these features, e.g. by composing the builders?

I see that in main branch MultiNamespacedCacheBuilder is deprecated, and namespacing will be achieved via cache.Options so the conflict seems to be removed. Do you have initial release plans for the new API?

@alvaroaleman
Copy link
Member

No, this isn't supported as of today. There is #2261 which desribes how and what we want to be configurable in the future. Could you have a look there and check if it would fulfill your use-case, please?

@tsaarni
Copy link
Author

tsaarni commented Apr 13, 2023

Thanks @alvaroaleman! I've checked the design doc and it looks good to me. I believe it will work fine for our use case. 👍

@tsaarni tsaarni closed this as completed Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants