mirror of
https://github.com/scylladb/scylladb.git
synced 2026-04-27 11:55:15 +00:00
e363febeb1
This patch adds reproducing tests for wrong handling of LIMIT in a query which uses a secondary index *and* filtering, described in issue #10649. In that case, Scylla incorrectly limits the number of rows found in the index *before* the filtering, while it should limit the number of rows *after* the filtering. The tests in this patch (which xfail on Scylla, and pass on Cassandra) go beyond the minimum required to reproduce this bug. It turns out that there are different sub-cases of this problem that go through different code paths, namely whether the base table has clustering keys or just partition keys, and whether the overall LIMITed result spans more than one page. So these tests attempt to also cover all these sub-cases. Without all these test sub-cases, an incomplete and incorrect fix of this bug may, by chance, cause the original test to succeed. Refs #10649 Signed-off-by: Nadav Har'El <nyh@scylladb.com> Closes #10658