apache / apache/druid

should we deprecate fieldExpression and extractionFn field in various Aggregators, DimensionSpecs, Filters etc

Open
#8,242 6 comments 0 reactions 0 assignees View on GitHub
Area - Querying Discuss
Dominant language
Java
Stars
14.1k
Forks
3.8k
Avg merge
2d 58m
Merged PRs (30d)
233

Description

they were added before `VirtualColumn` was introduced and I think a better alternative now could be to have a extractionFn based `VirtualColumn` that could be used and expression based virtual column exists already.
That way, all the filter and aggregator implementations don't need to handle them in different places.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the Aggregators, DimensionSpecs, and Filters that currently expose fieldExpression or extractionFn, then compare them with VirtualColumn and the existing expression-based virtual column. Done means reaching and documenting a project decision on whether these fields can be deprecated and replaced through VirtualColumn without leaving inconsistent handling across implementations.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
databases
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.