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

Add support for secondary dimensions #6021

Closed
wants to merge 2 commits into from

Conversation

nikkhils
Copy link
Contributor

@nikkhils nikkhils commented Aug 29, 2023

Allow users to specify a specific column to be used as a secondary dimension using the add_dimension() API. A secondary dimension does not participate in partitioning of the data. Such a secondary dimension will be used for chunk pruning if the WHERE clause of a SQL query specifies ranges on such a column.

Disable-check: force-changelog-file

Disable-check: commit-count

@nikkhils nikkhils self-assigned this Aug 29, 2023
@github-actions github-actions bot requested review from akuzm and pmwkaa August 29, 2023 12:03
@github-actions
Copy link

@pmwkaa, @akuzm: please review this pull request.

Powered by pull-review

@nikkhils nikkhils marked this pull request as draft August 29, 2023 12:03
@nikkhils nikkhils force-pushed the secondary_dims branch 4 times, most recently from 2ea9d83 to 49f9056 Compare September 4, 2023 07:37
@codecov
Copy link

codecov bot commented Sep 4, 2023

Codecov Report

Merging #6021 (7f9032c) into main (44e41c1) will decrease coverage by 0.10%.
The diff coverage is 51.13%.

@@            Coverage Diff             @@
##             main    #6021      +/-   ##
==========================================
- Coverage   81.38%   81.29%   -0.10%     
==========================================
  Files         243      243              
  Lines       55940    56011      +71     
  Branches    12385    12399      +14     
==========================================
+ Hits        45526    45532       +6     
- Misses       8088     8108      +20     
- Partials     2326     2371      +45     
Files Changed Coverage Δ
src/ts_catalog/catalog.h 100.00% <ø> (ø)
src/dimension_slice.c 85.94% <27.77%> (-2.19%) ⬇️
src/hypertable.c 84.62% <33.33%> (-0.13%) ⬇️
src/dimension.c 89.65% <43.90%> (-2.03%) ⬇️
src/chunk.c 89.81% <75.00%> (-0.04%) ⬇️
src/chunk_constraint.c 91.59% <81.81%> (-0.46%) ⬇️

... and 47 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Allow users to specify a specific column to be used as a secondary
dimension using the add_dimension() API. A secondary dimension does not
participate in partitioning of the data. Such a secondary dimension
will be used for chunk pruning if the WHERE clause of a SQL query
specifies ranges on such a column.
Create dimension_slide and chunk_constraint entries for chunks
belonging to a hypertable which has secondary dimensions on it. The
dimension slice entry will have -inf/+inf as start/end range for this
secondary dimension and the chunk_constraint entry will refer back to
it.

This start/end range will be refreshed later.
@nikkhils
Copy link
Contributor Author

nikkhils commented Mar 4, 2024

Closing this. A newer draft PR #6727 covers this same functionality now.

@nikkhils nikkhils closed this Mar 4, 2024
@nikkhils nikkhils deleted the secondary_dims branch March 4, 2024 10:56
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

Successfully merging this pull request may close these issues.

None yet

1 participant