delta-io/delta

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

Open

#2,416 创建于 2023年12月28日

在 GitHub 查看
 (2 评论) (0 反应) (0 负责人)Scala (8,807 star) (2,100 fork)batch import
enhancementgood first issuekernel

描述

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.

贡献者指南

技术栈
scala
领域
database
议题类型
feature
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
half day
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
fresh
清晰度议题是否清楚说明期望改动、验收标准和下一步。
clear
前置要求
Familiarity with Delta KernelUnderstanding of PR #2415
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70
研究方向
Examine the changes in PR #2415 that added COALESCE support for boolean types. Identify the check in DefaultExpressionEvaluator that restricts COALESCE to boolean types. Add tests for other data types (e.g., integer, string) and remove the boolean only check. Refer to the existing test patterns in the Kernel module.