Code review comment for lp:~radix/storm/count-limited-union

Revision history for this message
Thomas Herve (therve) wrote :

The message is definitely outdated, by only mentioning __contains__, but it seems to me that's it's still valid: the order_by value is not passed along, so it can't be used by the new expression? Your test doesn't check that as there is only one value.

« Back to merge proposal