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

Make scheduledRefreshTimeZones aware of the security context #5015

Closed
igorlukanin opened this issue Jul 28, 2022 · 1 comment
Closed

Make scheduledRefreshTimeZones aware of the security context #5015

igorlukanin opened this issue Jul 28, 2022 · 1 comment
Labels
enhancement New feature proposal pre-aggregations Issues related to pre-aggregations

Comments

@igorlukanin
Copy link
Member

Is your feature request related to a problem? Please describe.
Currently, scheduledRefreshTimeZones accepts an array of time zones. Thus, in a multitenant context, pre-aggregations for all tenants will be multiplied by the number of specified time zones.

Describe the solution you'd like
scheduledRefreshTimeZones should also accept a function (which, in turn, will accept securityContext as an argument) and return an array of time zones.

Describe alternatives you've considered

Additional context
This was requested at the recent workshop.

@igorlukanin igorlukanin added enhancement New feature proposal pre-aggregations Issues related to pre-aggregations labels Jul 28, 2022
@igorlukanin
Copy link
Member Author

Duplicates #3509

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature proposal pre-aggregations Issues related to pre-aggregations
Projects
None yet
Development

No branches or pull requests

1 participant