delta-io/delta

[Kernel][Defaults] Support `COALESCE` for all data types in the `DefaultExpressionHandler`

Open

#2416 aperta il 28 dic 2023

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Scala (2100 fork)batch import
enhancementgood first issuekernel

Metriche repository

Star
 (8807 star)
Metriche merge PR
 (Merge medio 7g 1h) (142 PR mergiate in 30 g)

Descrizione

Feature request

Which Delta project/connector is this regarding?

  • Spark
  • Standalone
  • Flink
  • Kernel
  • Other (fill in here)

Overview

#2415 adds support for COALESCE for boolean type expressions. We should support all data types.

Further details

#2415 already adds the needed changes. To support more data types we just need to add tests and remove the check that allows only boolean type in DefaultExpressionEvaluator.

Guida contributor

Direzione di ricerca
Esamina le modifiche nella PR #2415 che hanno aggiunto il supporto COALESCE per i tipi booleani. Identifica il controllo in DefaultExpressionEvaluator che limita COALESCE ai soli tipi booleani. Aggiungi test per altri tipi di dati (ad es. intero, stringa) e rimuovi il controllo limitato ai booleani. Fai riferimento ai modelli di test esistenti nel modulo Kernel.
Tech stack
scala
Dominio
backenddatabase
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ScalaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60