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

Enable partition_by config value for columns that are not lineages #597

Open
Taepper opened this issue Sep 25, 2024 · 0 comments
Open

Enable partition_by config value for columns that are not lineages #597

Taepper opened this issue Sep 25, 2024 · 0 comments
Labels
enhancement New feature or request question Further information is requested

Comments

@Taepper
Copy link
Collaborator

Taepper commented Sep 25, 2024

Currently, the database_config's field partition_by must refer to a lineage column. This could be generalized to allow all data types. The implementation effort should be marginal.

Yet, it is undecided whether we want to support that functionality.

Pro:
We can enable clearer and better to debug partitioning logic, which can be fine-tuned by a silo user.

Con:
While the code might not become more complicated, we still have a bigger surface for which we need testing

@fengelniederhammer fengelniederhammer added enhancement New feature or request question Further information is requested labels Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants