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

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
Next Next commit
add test
  • Loading branch information
@kgyrtkirk
kgyrtkirk committed Jun 5, 2024
commit 31ffbba01522281581fa4c801a35acff836e64fd
12 changes: 12 additions & 0 deletions sql/src/test/java/org/apache/druid/sql/calcite/CalciteQueryTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1746,6 +1746,18 @@ public void testAnyAggregatorsSkipNullsWithFilter()
);
}

@Test
public void testOrderByEarliestFloatSubQuery()
{
testBuilder()
.sql(
"select dim1,e "
+ "from (SELECT dim1, EARLIEST(f1) e FROM druid.numfoo GROUP BY 1 ORDER BY 2 LIMIT 10) t "
+ "order by 1 limit 2"
)
.run();
}

@Test
public void testOrderByEarliestFloat()
{
Expand Down