Skip to content

Commit a8cd1f9

Browse files
committed
drop support for CockroachDB 23.1
1 parent 4c9541f commit a8cd1f9

File tree

3 files changed

+1
-36
lines changed

3 files changed

+1
-36
lines changed

.github/workflows/tests.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,11 +29,6 @@ jobs:
2929
fail-fast: false
3030
matrix:
3131
include:
32-
- crdb-version: v23.1.30
33-
- crdb-version: v23.1.30
34-
use_psycopg2: psycopg2
35-
- crdb-version: v23.1.30
36-
use_server_side_binding: server_side_binding
3732
- crdb-version: v23.2.21
3833
- crdb-version: v23.2.21
3934
use_psycopg2: psycopg2

README.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,3 @@ using back to Cockroach Labs. To disable this, set
147147
`unimplemented: table <...> is currently undergoing a schema change` if a
148148
later operation tries to modify the table before the asynchronous query
149149
finishes.
150-
151-
## Known issues and limitations in CockroachDB 23.1.x and earlier
152-
153-
- CockroachDB doesn't support by ordering by JSON.

django_cockroachdb/features.py

Lines changed: 1 addition & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66

77
class DatabaseFeatures(PostgresDatabaseFeatures):
8-
minimum_database_version = (23, 1)
8+
minimum_database_version = (23, 2)
99

1010
# Cloning databases doesn't speed up tests.
1111
# https://github.com/cockroachdb/django-cockroachdb/issues/206
@@ -75,10 +75,6 @@ def introspected_field_types(self):
7575
'virtual': None,
7676
}
7777

78-
@cached_property
79-
def is_cockroachdb_23_2(self):
80-
return self.connection.cockroachdb_version >= (23, 2)
81-
8278
@cached_property
8379
def is_cockroachdb_24_1(self):
8480
return self.connection.cockroachdb_version >= (24, 1)
@@ -201,28 +197,6 @@ def django_test_expected_failures(self):
201197
# concat(): unknown signature: concat(string, int2) (desired <string>)
202198
'db_functions.text.test_concat.ConcatTests.test_concat_non_str',
203199
})
204-
if not self.is_cockroachdb_23_2:
205-
expected_failures.update({
206-
# cannot index a json element:
207-
# https://github.com/cockroachdb/cockroach/issues/35706
208-
'schema.tests.SchemaTests.test_func_index_json_key_transform',
209-
# ordering by JSON isn't supported:
210-
# https://github.com/cockroachdb/cockroach/issues/35706
211-
'db_functions.comparison.test_json_object.JSONObjectTests.test_order_by_key',
212-
'db_functions.comparison.test_json_object.JSONObjectTests.test_order_by_nested_key',
213-
'expressions_window.tests.WindowFunctionTests.test_key_transform',
214-
'model_fields.test_jsonfield.TestQuerying.test_deep_distinct',
215-
'model_fields.test_jsonfield.TestQuerying.test_order_grouping_custom_decoder',
216-
'model_fields.test_jsonfield.TestQuerying.test_ordering_by_transform',
217-
'model_fields.test_jsonfield.TestQuerying.test_ordering_grouping_by_key_transform',
218-
# unsupported comparison operator: <jsonb> > <string>:
219-
# https://github.com/cockroachdb/cockroach/issues/49144
220-
'model_fields.test_jsonfield.TestQuerying.test_deep_lookup_transform',
221-
# DataError: incompatible COALESCE expressions: expected pi() to be
222-
# of type decimal, found type float
223-
# https://github.com/cockroachdb/cockroach/issues/73587#issuecomment-988408190
224-
'aggregation.tests.AggregateTestCase.test_aggregation_default_using_decimal_from_database',
225-
})
226200
if self.is_cockroachdb_24_1:
227201
# USING cast required: https://github.com/cockroachdb/cockroach/issues/82416#issuecomment-2029803229
228202
expected_failures.update({

0 commit comments

Comments
 (0)