This is great! It reduced the runtime of single tests in Presto from 25s to 6s which is huge for me because I run these all day while developing.
I made a few small style changes and simplified the map build in distribution. it turned out we don't need the binary search at all since the values in the map are in order.
This is great! It reduced the runtime of single tests in Presto from 25s to 6s which is huge for me because I run these all day while developing.
I made a few small style changes and simplified the map build in distribution. it turned out we don't need the binary search at all since the values in the map are in order.
The change is merged. Again thanks!