-
Notifications
You must be signed in to change notification settings - Fork 230
Topn fill #57
Comments
@tgruben Yep, I can take a look. |
@tgruben Did the previous implementation do a refetch? From what I see in 6822dd9 it does a This is the only change required for I could see doing a min Here are the lines from the previous implementation I was referencing: https://github.com/umbel/pilosa/blob/6822dd99b5245195dbe45fbe9cb1718fdfb51a2f/core/topn.go#L174 |
Yes,but most of the code that does the fill in was located in it is basically the results of the first scatter Does that code make sense or do I need to expand..that one function is -Todd On Mon, Feb 29, 2016 at 5:30 PM, Ben Johnson notifications@github.com
|
@tgruben Yep, that make sense. I missed that refetching part the first time around. I'll get that added in. |
BLOCKED: Pilosa update
return total match counts for either min or max
The topn needs a bit of adjustment. The process should be:
scatter out topn to each slice
gather results
-->fetch missing counts from non reporting slices<-- the missing part
sort and return top n
I've added a test, TestExecutor_Execute_TopN_fill, to express the problem
you think you can fix that @benbjohnson ?
The text was updated successfully, but these errors were encountered: