delta-io/delta

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

Chiusa

#2416 aperta il 28 dic 2023

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Scala (2159 fork)batch import
enhancementgood first issuekernelstale

Metriche repository

Star
 (8950 stelle)
Metriche merge PR
 (Metriche PR in attesa)

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