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

Remove partition columns from Z Order optimization in io.py #244

Open
R7L208 opened this issue Aug 15, 2022 · 2 comments
Open

Remove partition columns from Z Order optimization in io.py #244

R7L208 opened this issue Aug 15, 2022 · 2 comments
Assignees
Labels
enhancement New feature or request question Further information is requested

Comments

@R7L208
Copy link
Contributor

R7L208 commented Aug 15, 2022

In io.py we Z ORDER on partitionCols + optimizationCols when useDeltaOpt is True. Since we can partition prune without Z Ordering on partition columns, I believe it makes sense to remove them from the Z Order clause to only optimize on optimizationCols if they are provided.

Is there another advantage to including partition columns within Z ORDER for time series other than data skipping?

@R7L208 R7L208 added bug Something isn't working question Further information is requested labels Aug 15, 2022
@R7L208 R7L208 added enhancement New feature or request and removed bug Something isn't working labels Aug 26, 2022
@rportilla-databricks
Copy link
Contributor

rportilla-databricks commented Oct 11, 2022 via email

@rportilla-databricks
Copy link
Contributor

This will be resolved when streaming AS OF joins are merged.

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

3 participants