apache / apache/pinot

Improve error message when aggregate function is applied to a nun-numeric column

Open
#8,290 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
6.1k
Forks
1.5k
Avg merge
1d 21h
Merged PRs (30d)
189

Description

Pinot currently supports aggregation functions for e.g max() on numeric values only, however if you run a query where a non-numeric column is passed as a value then error message generated is really vague and totally misleading:

Screen Shot 2022-03-03 at 12 17 15 PM

We should have a explicit error message indicating lack of support for non-numeric data and the offending function used in the query.

cc @mayankshriv @kishoreg

Contributor guide

Open the contributing guide

Research direction

Start by reproducing a Pinot query that applies max() to a non-numeric column, then trace the aggregate-function error handling. Done means the error explicitly states that non-numeric data is unsupported and identifies the offending aggregation function.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
databases
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.