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

Sotirissl/new decompression workers #5696

Closed
wants to merge 1 commit into from
Closed

Sotirissl/new decompression workers #5696

wants to merge 1 commit into from

Conversation

sotirissl
Copy link
Contributor

Improve parallel workers for decompression

Planner may also choose parallel plans when decompressing chunks.
Can use for decompression as many workers wants per chunk.

Make use of PostgreSQL function
create_plain_partial_paths()
for create parallel plans

Co-authored-by: Jan Kristof Nidzwetzki [email protected]

@codecov
Copy link

codecov bot commented May 18, 2023

Codecov Report

Merging #5696 (50a27d3) into main (8e69a99) will decrease coverage by 10.79%.
The diff coverage is 100.00%.

❗ Current head 50a27d3 differs from pull request most recent head 7f37842. Consider uploading reports for the commit 7f37842 to get more accurate results

@@             Coverage Diff             @@
##             main    #5696       +/-   ##
===========================================
- Coverage   90.98%   80.20%   -10.79%     
===========================================
  Files         230      231        +1     
  Lines       54473    48174     -6299     
  Branches        0    11991    +11991     
===========================================
- Hits        49564    38636    -10928     
+ Misses       4909     4269      -640     
- Partials        0     5269     +5269     
Impacted Files Coverage Δ
src/import/allpaths.c 68.24% <100.00%> (-12.38%) ⬇️
tsl/src/nodes/decompress_chunk/decompress_chunk.c 83.55% <100.00%> (-10.97%) ⬇️

... and 218 files with indirect coverage changes

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

Planner may also choose parallel plans when decompressing chunks.
Can use for decompression as many workers wants per chunk.

Co-authored-by: Jan Kristof Nidzwetzki <[email protected]>
@sotirissl
Copy link
Contributor Author

Pushed and Merged with Improve parallel workers for decompression #5655
#5655

@sotirissl sotirissl closed this Jun 2, 2023
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.

1 participant