-
Notifications
You must be signed in to change notification settings - Fork 31
Intra partition parallelism #346
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
Intra partition parallelism #346
Conversation
…ssing and code clarity
…into intra-partition-parallelism
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #346 +/- ##
=========================================
+ Coverage 0.78% 0.95% +0.16%
=========================================
Files 107 108 +1
Lines 25995 26278 +283
Branches 17150 17306 +156
=========================================
+ Hits 204 250 +46
- Misses 25590 25812 +222
- Partials 201 216 +15 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…tra-partition-parallelism
… to the 25 allowed.
src/query/processor/executor/IntraPartitionParallelExecutor.cpp
Outdated
Show resolved
Hide resolved
src/query/processor/executor/IntraPartitionParallelExecutor.cpp
Outdated
Show resolved
Hide resolved
miyurud
left a comment
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.
I have put some comments. Please address all of those and reply to each comment how you addressed those.
miyurud
left a comment
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.
I have put some comments. Please address them all and reply each comment how you addressed them.
|


No description provided.