-
Notifications
You must be signed in to change notification settings - Fork 848
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
Don't look up entire Chunk struct for compressed chunks #5970
Conversation
@konskov, @pmwkaa: please review this pull request.
|
Codecov Report
@@ Coverage Diff @@
## main #5970 +/- ##
==========================================
+ Coverage 79.52% 87.00% +7.48%
==========================================
Files 243 243
Lines 49266 55974 +6708
Branches 12346 12385 +39
==========================================
+ Hits 39177 48702 +9525
- Misses 4225 4910 +685
+ Partials 5864 2362 -3502
... and 216 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@@ -72,7 +73,8 @@ static DecompressChunkPath *decompress_chunk_path_create(PlannerInfo *root, Comp | |||
int parallel_workers, | |||
Path *compressed_path); | |||
|
|||
static void decompress_chunk_add_plannerinfo(PlannerInfo *root, CompressionInfo *info, Chunk *chunk, | |||
static void decompress_chunk_add_plannerinfo(PlannerInfo *root, Hypertable *ht, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems the new Hypertable *ht
parameter is not used in this function.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a minor comment, rest looks good to me.
It's not needed. Also add them to baserel cache.
f98eaf0
to
00b0e83
Compare
It's not needed. Also add them to baserel cache.
Disable-check: force-changelog-file