Add tests for IN-list filter pruning and NOT filter behavior
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 47/100
Rechercherichtung
Start by locating the TestFilterPushdown suite and the Rust in_list_excludes() method mentioned in the issue. Add regression coverage for IN-list partition pruning and both NOT comparison cases, using the four-partition example as a guide. Done means the tests verify that only partitions whose ranges may contain matching values are read.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Problem
The TestFilterPushdown suite is missing coverage for two cases:
1. IN-list pruning
The Rust in_list_excludes() method is implemented but never exercised by tests. A test should verify that WHERE dim IN (v1, v2) prunes partitions where none of v1, v2 fall within the partition range.
Example: 4 time partitions, WHERE time IN ('2020-01-05', '2020-01-10') should read only partition 0.
2. NOT filter behavior
The Expr::Not arm previously had a correctness bug (now fixed). A regression test should verify:
NOT (col > X)correctly includes partitions containing values ≤ XNOT (col < X)correctly includes partitions containing values ≥ X
Why now
Adding these tests serves as a regression guard to confirm that the Expr::Not fix remains correct and that IN-list pruning works as advertised. Without them, future changes to the pruning logic could silently regress.
- Vorherrschende Sprache
- Python
- Sterne
- 142
- Forks
- 19
- Ø Merge
- 1 T. 6 Std.
- Gemergte PRs (30 T.)
- 10
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus xqlsystems/xarray-sql
-
documentation
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 85/100
xqlsystems/xarray-sql#268 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 64/100
xqlsystems/xarray-sql#267 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 74/100
xqlsystems/xarray-sql#243 ·
Maintainer antworten meist innerhalb von 1 Tag
-
publish.yml: the TestPyPI verification step is unreachable (and installs the wrong package name)Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
xqlsystems/xarray-sql#242 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
xqlsystems/xarray-sql#237 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in xqlsystems/xarray-sql
Ähnliche Issues
-
[oblt-aw][autodoc] Update workflow docs for gh-aw v0.92.5 and detector permission scopeEvtl. vergeben @github-actions hat das heute übernommen. Offen
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 88/100
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag
-
feat(ide): VS Code access through the shipped MCP server (config doc and tool-list smoke test)Offenroadmap-100
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag
-
comp/tools P2 tool/browser tool/file type/bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
NousResearch/hermes-agent#143418 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
python-hyper/uritemplate#164 ·