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

Security Context And Timezone specific Pre aggregations #7533

Closed
pranav-loops opened this issue Dec 14, 2023 · 2 comments
Closed

Security Context And Timezone specific Pre aggregations #7533

pranav-loops opened this issue Dec 14, 2023 · 2 comments
Labels
enhancement New feature proposal pre-aggregations Issues related to pre-aggregations

Comments

@pranav-loops
Copy link

We can build pre aggregations for specific time zones using https://cube.dev/docs/reference/rest-api#triggering-jobs but this is not possible with the refresh worker.

Say i have SecurityContext1 which i want to use with Timezone1 and SecurityContext2 with Timezone2, the only way to ensure the preaggregations are built by the refresh worker is to provide

scheduledRefreshTimeZones: ['timezone1', 'timezone2'],

This solves the use case but creates additional pre aggregations which are never used - ie. SecurityContext1, Timezone2 and SecurityContext2, Timezone1

In our case certain security contexts only need to work with specific timezones, but as of now the current configuration provided from the refresh worker, all time zones apply to all security contexts which seems like an overhead too. Is there a way to control this?

Maybe allowedTimezones or defaultTimezone could be a property within the securityContext? Is there any other way to configure different security contexts with different timezones from the refresh worker?

@pranav-loops pranav-loops added the question The issue is a question. Please use Stack Overflow for questions. label Dec 14, 2023
@igorlukanin
Copy link
Member

Hi @pranav-loops 👋

I think that making scheduledRefreshTimeZones aware of the security context would be the right move here. That way, you would be able to configure specific time zones for specific security contexts—and build only necessary pre-aggregations.

@igorlukanin igorlukanin added enhancement New feature proposal pre-aggregations Issues related to pre-aggregations and removed question The issue is a question. Please use Stack Overflow for questions. labels Jan 3, 2024
@igorlukanin
Copy link
Member

Let me close this one in favor of #5015—but please feel free to follow that issue.

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

2 participants