apache / apache/datafusion

Write tests for streaming_merge

Aperta
#6,907 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
enhancement
Lingua principale
Rust
Stelle
9.3k
Fork
2.4k
Merge medio
3g 11h
PR unite (30g)
360

Descrizione

### 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_

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia in datafusion-examples/examples e esamina i file esistenti merge_fuzz.rs e sort.rs per individuare pattern che possano esercitare l’API pubblica physical_plan::merge::streaming_merge. Aggiungi un esempio o un test che ne dimostri l’uso e conferma che il test preservi la copertura dell’API pubblica.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
rust
Ambito
data-engineering, testing
Tipo di issue
Funzionalità
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
55/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.