Make ScalarValue an ArrayRef Wrapper
- 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