delta-io/delta

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

Open

#2.416 aberto em 28 de dez. de 2023

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)Scala (2.100 forks)batch import
enhancementgood first issuekernel

Métricas do repositório

Stars
 (8.807 stars)
Métricas de merge de PR
 (Mesclagem média 7d 1h) (142 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Examine as alterações no PR #2415 que adicionaram suporte a COALESCE para tipos booleanos. Identifique a verificação em DefaultExpressionEvaluator que restringe COALESCE apenas a tipos booleanos. Adicione testes para outros tipos de dados (por exemplo, inteiro, string) e remova a verificação restrita a booleanos. Consulte os padrões de teste existentes no módulo Kernel.
Pilha de tecnologia
scala
Domain
backenddatabase
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ScalaGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60