Closed paisible-wanderer closed 5 years ago
@zverok & @v0dro: ping!
@paisible-wanderer I'm extremely sorry for this huge delay. I'll look into this PR ASAP. Just give me a day more since the changes are many.
OK I just went through your changes and they're pretty amazing. I'm merging. Thank you very much for your changes and hope to see more of these sometime again soon :)
And sorry again for the delay. The PR just went past my checks for some reason.
@v0dro thanks!
Hello,
In the continuation of #454, I have made some improvements likely impacting all the aggregation related code (by making things "lazy").
Bellow are some aggregation benchmarks for different ratios of rows vs number of groups. (Summary: some things are ~10 times faster).
=========================
Please, let me know if there is anything I can improve.