Skip to content

Commit

Permalink
tests: xfail test_order_by_success_case for scylla
Browse files Browse the repository at this point in the history
test is failing with the following:
```
cassandra.InvalidRequest: Error from server: code=2200 [Invalid query]
message="Order by currently only supports the ordering of columns following their declared order in the PRIMARY KEY"
```

seems like scylla doesn't support ordering by clustering keys

Ref: scylladb#343
  • Loading branch information
fruch committed Jul 4, 2024
1 parent c15ba8e commit d9e3116
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tests/integration/cqlengine/query/test_queryset.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
from cassandra.util import uuid_from_time
from cassandra.cqlengine.connection import get_session
from tests.integration import PROTOCOL_VERSION, CASSANDRA_VERSION, greaterthancass20, greaterthancass21, \
greaterthanorequalcass30, TestCluster, requires_collection_indexes
greaterthanorequalcass30, TestCluster, requires_collection_indexes, xfail_scylla
from tests.integration.cqlengine import execute_count, DEFAULT_KEYSPACE


Expand Down Expand Up @@ -600,6 +600,7 @@ def test_distinct_with_explicit_count(self):
@requires_collection_indexes
class TestQuerySetOrdering(BaseQuerySetUsage):
@execute_count(2)
@xfail_scylla(reason="Scylla does not support ordering on non-primary key columns: https://github.com/scylladb/python-driver/issues/343")
def test_order_by_success_case(self):
q = TestModel.objects(test_id=0).order_by('attempt_id')
expected_order = [0, 1, 2, 3]
Expand Down

0 comments on commit d9e3116

Please sign in to comment.