Improve consistency of expression names
Nessuno ha ancora preso questa issue.
- Lingua principale
- Rust
- Stelle
- 9.3k
- Fork
- 2.4k
- Merge medio
- 3g 6h
- PR unite (30g)
- 363
Descrizione
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
We have many different ways to create names from expressions with duplicated and sometimes inconsistent code.
Logical Expression:
- Display trait
- Debug trait
Expr.name()method (wrapper forcreate_namefunction)ExprIdentifierVisitor::desc_expr
Physical Expression:
- Display trait
- Debug trait
create_physical_namefunction
One example of confusion is that queries sometimes result in field names containing Divide and sometimes /. For example:
decimal_simple.c1 / CAST(Float64(0.00001) AS Decimal128(5, 5))uses/CAST(decimal_simple.c1 AS Decimal128(30, 19)) Divide CAST(decimal_simple.c5 AS Decimal128(30, 19))usesDivide
Describe the solution you'd like
Make names more consistent and avoid duplicate code
Describe alternatives you've considered
None
Additional context
None
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia tracciando i percorsi di denominazione elencati nell’issue: le implementazioni di Display e Debug, Expr.name(), create_name, ExprIdentifierVisitor::desc_expr e create_physical_name. Confronta la loro gestione di operatori come Divide e /; il lavoro è completato quando i nomi delle espressioni sono coerenti e la logica di denominazione duplicata è ridotta.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- rust
- Ambito
- data-engineering
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 30/100