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
-
follow-up
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Maintainer antworten meist innerhalb von 1 Tag
-
Download helper still uses huggingface_hub arguments removed in 1.0: USE_SYMLINKS is silently ignored and every download ends with a WinError 3 lineEvtl. vergeben @li-lizhe hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
IPv6 node addresses produce an invalid request URLEvtl. vergeben @RaphaelFakhri hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
typesense/typesense-python#154 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
elastic/elasticsearch-py#3592 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 77/100
developmentseed/titiler-cmr#250 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag