apache / apache/datafusion

Make ScalarValue an ArrayRef Wrapper

Aperta
#7,353 14 commenti 2 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?

We have recently standardised a scalar abstraction upstream, based around the existing Array abstractions.

The rationale behind this is expanded upon here https://github.com/apache/arrow-rs/pull/4393#issue-1750177364

### Describe the solution you'd like

I would like ScalarValue to just be a wrapper around an `ArrayRef`. This would avoid a huge amount of duplicated and/or boilerplate code, whilst also discouraging implementing non-vectorizable kernels based around ScalarValue.

### 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 leggendo le astrazioni ScalarValue e ArrayRef e la motivazione nella pull request upstream collegata. Mappa il codice e i test interessati dalla sostituzione di ScalarValue con un wrapper ArrayRef, quindi definisci il completamento come il mantenimento del comportamento esistente rimuovendo al contempo le implementazioni duplicate specifiche degli scalari.

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

Valutazione

Stack tecnologico
rust
Ambito
backend, data
Tipo di issue
Funzionalità
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
20/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.