delta-io/delta

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

Open

#2 416 ouverte le 28 déc. 2023

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Scala (2 100 forks)batch import
enhancementgood first issuekernel

Métriques du dépôt

Stars
 (8 807 stars)
Métriques de merge PR
 (Merge moyen 7j 1h) (142 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez les modifications de la PR #2415 qui ont ajouté la prise en charge de COALESCE pour les types booléens. Identifiez la vérification dans DefaultExpressionEvaluator qui restreint COALESCE aux types booléens uniquement. Ajoutez des tests pour d'autres types de données (par exemple, entier, chaîne) et supprimez la vérification limitée aux booléens. Référez vous aux modèles de test existants dans le module Kernel.
Stack technique
scala
Domaine
backenddatabase
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ScalaGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60