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.

Sign up for GitHub

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

Jump to bottom

GroupBy query not multi-threaded? #96

Closed
cheddar opened this issue Mar 4, 2013 · 19 comments
Closed

GroupBy query not multi-threaded? #96

cheddar opened this issue Mar 4, 2013 · 19 comments
Milestone
0.4.32 and before

Comments

@cheddar
Copy link
Contributor

cheddar commented Mar 4, 2013

The GroupBy Query is not running in multiple threads because of the mergeRunners() method on GroupByQueryRunnerFactory.

We should adjust it with a configuration switch to switch between using one thread or multiple and have it default to using multiple.
@darxriggs
Copy link
Contributor

+1

16 similar comments
@wolfs
Copy link

wolfs commented Apr 7, 2013

+1

@lepokle
Copy link

lepokle commented Apr 7, 2013

+1

@gerhard-mueller
Copy link

+1

@wuan
Copy link

wuan commented Apr 7, 2013

+1

@Achimh3011
Copy link

+1

@wolfgangmuender
Copy link

+1

@stockc
Copy link

stockc commented Apr 7, 2013

+1

@code-acrobat
Copy link

+1

@moritzp
Copy link

moritzp commented Apr 8, 2013

+1

@kalleb
Copy link

kalleb commented Apr 8, 2013

+1

@andrgrau
Copy link

andrgrau commented Apr 8, 2013

+1

@tom-mi
Copy link

tom-mi commented Apr 8, 2013

+1

@MrShine
Copy link

MrShine commented Apr 8, 2013

+1

@rincedd
Copy link

rincedd commented Apr 8, 2013

+1

@tosts
Copy link

tosts commented Apr 8, 2013

+1

@codecholeric
Copy link

+1

@cheddar
Copy link
Contributor Author

cheddar commented Apr 8, 2013

Awesome guys. I'm working on this, will check in a fix soon :).

@darxriggs
Copy link
Contributor

I cannot wait to see this in action. :-)

b-slim pushed a commit to b-slim/druid that referenced this issue Jul 30, 2015
Merge pull request apache#96 from guobingkun/metamx
64a4fa2 
Sync
maytasm pushed a commit to maytasm/druid that referenced this issue May 20, 2020
@zachjsh
druid.storage.maxListingLength should default to 1000 for s3 (apache#…
231ab16 
…9858) (apache#96)

* druid.storage.maxListingLength should default to 1000 for s3

* * Address review comments

* * Address review comments

* * Address comments
abhishekagarwal87 referenced this issue in abhishekagarwal87/druid Dec 14, 2020
@maytasm
Merge pull request #96 from implydata/resolve-merge-conflict
1c9fda0 
Resolve merge conflict
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.4.32 and before
Development

No branches or pull requests