New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add experimental support for first/last for double/float/long #10702 #14462

Merged

cryptoe merged 17 commits into apache:master from ankit0811:feature_FirstLast_DoubleFLoatLong

Dec 12, 2023

Merged

Add experimental support for first/last for double/float/long #10702 #14462

addressing comments + extending LongString to the abstract classes

This check has been archived and is scheduled for deletion. Learn more about checks retention

GitHub Advanced Security / CodeQL succeeded Dec 8, 2023 in 4s

2 new alerts

New alerts in code changed by this pull request

2 warnings

See annotations below for details.

View all branch alerts.

Annotations

Check warning on line 161 in processing/src/main/java/org/apache/druid/query/aggregation/first/NumericFirstVectorAggregator.java

Code scanning / CodeQL

Dereferenced variable may be null Warning

Variable

inPair

may be null at this access as suggested by

this

null guard.

Check warning on line 166 in processing/src/main/java/org/apache/druid/query/aggregation/last/NumericLastVectorAggregator.java

Code scanning / CodeQL

Dereferenced variable may be null Warning

Variable

inPair

may be null at this access as suggested by

this

null guard.