Skip to content

Commit 9779c9a

Browse files
chore: add missing concurrency group to CI workflow (#21)
1 parent 734607d commit 9779c9a

3 files changed

Lines changed: 6 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,10 @@ on:
1414
- main
1515
workflow_dispatch:
1616

17+
concurrency:
18+
group: ci-${{ github.head_ref || github.ref }}
19+
cancel-in-progress: true
20+
1721
env:
1822
POSTGRES_USER: postgres
1923
POSTGRES_PASSWORD: postgres

paradedb/sqlalchemy/indexing.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -791,6 +791,6 @@ def validate_pushdown(stmt: Any) -> list[str]:
791791
warnings.append("No ParadeDB predicate found in WHERE clause; query will not use a BM25 index")
792792

793793
if has_order_by(stmt) and not has_limit(stmt):
794-
warnings.append("ORDER BY is present without LIMIT; top-N pushdown to ParadeDB requires both")
794+
warnings.append("ORDER BY is present without LIMIT; Top K pushdown to ParadeDB requires both")
795795

796796
return warnings

tests/unit/test_validation_cache_unit.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ def test_validate_pushdown_ignores_limit_identifier_names():
172172

173173
warnings = validate_pushdown(stmt)
174174

175-
assert "ORDER BY is present without LIMIT; top-N pushdown to ParadeDB requires both" in warnings
175+
assert "ORDER BY is present without LIMIT; Top K pushdown to ParadeDB requires both" in warnings
176176

177177

178178
def test_custom_nodes_have_cache_keys():

0 commit comments

Comments
 (0)