Michał Siedlaczek

Let me explain why we want to use a pointer in `QueryContainer` (the same goes for the term resolver, although to a lesser extend, as in it's not quite as...

I thought about having threshold, and I agree that it's not ideal just having a threshold. I think we should have threshold**s**: ```json thresholds: [ {"k": 10, "score": 5.3}, {"k":...

@JMMackenzie do you think you'd be able to run a benchmark comparing this to master? I think one block codec should be enough to verify if there's any regression, unless...

Thanks @JMMackenzie can you try passing `-DPISA_ENABLE_IPO` to cmake, recompile and run again? I'm curious if this helps.

Actually, I see a bug there, I name the option with PISA_ but the check is without it (right below). You'd need to fix that. Whatever you do, please check...

You'll also need: ```cmake cmake_policy(SET CMP0069 NEW) # Use IPO (LTO) when requested ```

> Are we willing to take this performance hit? I'm happy to revisit these tests soon if we want to make sure... Possibly... but not entirely sure. Kind of want...

@fulmicoton Thanks! I noticed that many "union" type of queries are now faster with Lucene. Is that due to their recent implementation of BMW? Do you know? Is that a...

It looks good. Nothing else to add comes to mind at the moment. You wrote: $50$ million, so you make sure to remove the $ signs, but other than that,...

Nothing else comes to mind. @JMMackenzie ?