apache / apache/datafusion

Write tests for streaming_merge

Offen
#6,907 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Rust
Sterne
9.3k
Forks
2.4k
Ø Merge
3 T. 11 Std.
Gemergte PRs (30 T.)
360

Beschreibung

### Is your feature request related to a problem or challenge?

With #6874, `physical_plan::merge::streaming_merge` is now public.

Without some care, the change may be lost in some future refactoring (e.g. if the code is moved to another module, for example, that is not pub).

### Describe the solution you'd like

Write a test, perhaps as an example of how to use this API, in: https://github.com/apache/arrow-datafusion/tree/main/datafusion-examples/examples

Might be able to adapt some of the existing code in https://github.com/apache/arrow-datafusion/blob/main/datafusion/core/tests/fuzz_cases/merge_fuzz.rs or https://github.com/apache/arrow-datafusion/blob/main/datafusion/core/benches/sort.rs

### Describe alternatives you've considered

_No response_

### Additional context

_No response_

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne in datafusion-examples/examples und sieh dir die vorhandenen Dateien merge_fuzz.rs und sort.rs auf Muster an, mit denen die öffentliche physical_plan::merge::streaming_merge API ausgeübt werden kann. Füge ein Beispiel oder einen Test hinzu, der ihre Verwendung demonstriert, und bestätige, dass der Test die Abdeckung der öffentlichen API beibehält.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
rust
Bereich
data-engineering, testing
Issue-Typ
Feature
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.