Skip to content
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

Jump to bottom

Fix incorrect comparator usage in FinalizingFieldAccessPostAggregator #16555

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Fix incorrect comparator usage in FinalizingFieldAccessPostAggregator #16555

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
31ffbba
add test
kgyrtkirk Jun 5, 2024
bd47fe7
fix finalizingfieldaccesspostaggregator
kgyrtkirk Jun 5, 2024
7e0eb34
use iq test instead of CQT
kgyrtkirk Jun 5, 2024
32b3978
add serde for histogram/histogramvisual
kgyrtkirk Jun 7, 2024
2ced982
undo add type arg
kgyrtkirk Jun 7, 2024
d662a62
more correct type in histogram module
kgyrtkirk Jun 7, 2024
e456fa4
cleanup
kgyrtkirk Jun 7, 2024
f0a1112
add serde for histo
kgyrtkirk Jun 7, 2024
99d2977
Merge remote-tracking branch 'apache/master' into fix-finalize-field-…
kgyrtkirk Jun 10, 2024
f6cede6
make basic serde for tminmax
kgyrtkirk Jun 10, 2024
c74481a
permit nulls; restore stuff for unknown inputs
kgyrtkirk Jun 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
cleanup
  • Loading branch information
@kgyrtkirk
kgyrtkirk committed Jun 7, 2024
commit e456fa4d5f9b82b2f9d7e9a67d7357a324c9f780
2 changes: 1 addition & 1 deletion ...a/org/apache/druid/query/aggregation/histogram/ApproximateHistogramAggregatorFactory.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@
import org.apache.druid.query.aggregation.AggregatorFactoryNotMergeableException;
import org.apache.druid.query.aggregation.AggregatorUtil;
import org.apache.druid.query.aggregation.BufferAggregator;
import org.apache.druid.query.aggregation.HistogramAggregatorFactory;
import org.apache.druid.query.aggregation.ObjectAggregateCombiner;
import org.apache.druid.query.aggregation.VectorAggregator;
import org.apache.druid.query.cache.CacheKeyBuilder;
Expand All @@ -43,6 +42,7 @@
import org.apache.druid.segment.vector.VectorColumnSelectorFactory;

import javax.annotation.Nullable;

import java.nio.ByteBuffer;
import java.util.Collections;
import java.util.Comparator;
Expand Down
1 change: 0 additions & 1 deletion processing/src/main/java/org/apache/druid/jackson/AggregatorsModule.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,6 @@ public AggregatorsModule()
ComplexMetrics.registerSerde(Histogram.TYPE.getComplexTypeName(), new HistogramSerde());
ComplexMetrics.registerSerde(HistogramVisual.TYPE.getComplexTypeName(), new HistogramVisualSerde());


setMixInAnnotation(AggregatorFactory.class, AggregatorFactoryMixin.class);
setMixInAnnotation(PostAggregator.class, PostAggregatorMixin.class);

Expand Down
Loading