Improvement in implementation of “:count is 0”

[adamhholmes]

@Jici

So just to be sure I understand correctly, this will always perform a msearch request but return less data or it’s now performing a maggregate request?

Seemingly, it’s the former… it still performs an msearch (for some reason), but rather than returning the full data of the first 10 items (which is what it always did before in my testing), it now only retrieves the first item, and only the Unique ID of that item. So, depending on the size of the data it may well be a lot less data.

But that means it still costs 0.3 WU instead of 0.2 - which is still 50% more than it should be, and still 50% more than using the number 1 instead of the number 0.

@georgecollier

Meanwhile, if I do :count is (any positive integer), it does an aggregate request and costs 0.2 WU.

There’s still clearly a difference between :count is 0 and :count is {positive integer}.

:count is 0 first costs 0.3 WU and :count is {positive integer} costs 0.2 WU.

Why?

It’s even more specific than that, in that it’s the number 0, only as a literal value that causes the problem.

If it’s a variable (i.e. a custom state) it works fine and just performs an agg search.

@bek

To confirm, “:count is 0” should now be the go to for checking if a list is empty, is that correct?

Nope…

Even with this ‘fix’ count is 0 still costs exactly 50% more in WU than count < 1