Wrap LexRequirement, LexOrdering into their own struct
- 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?
Currently `LexOrdering` is a type alias for `Vec`. Similarly, `LexRequirement` is a type alias for `Vec`.
I think in the long run, it is better to have something like below struct
```rust
struct LexOrdering{
inner: Vec
}
```
With this in the future we can extend the type without changing all the types.
Similar thing can be done for `LexRequirement` also.
### Describe the solution you'd like
_No response_
### 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 individuando le definizioni e gli utilizzi di LexOrdering, LexRequirement, PhysicalSortExpr e PhysicalSortRequirement. Esamina come vengono costruiti i due alias e come vengono passati attraverso l’intero motore delle query prima di determinare l’API della struct. Il lavoro è completo quando entrambi gli alias sono stati sostituiti senza compromettere il loro comportamento esistente o i relativi chiamanti, e i test pertinenti continuano a passare.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- rust
- Ambito
- backend
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100