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.

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

ISPN-5788 Add more tests for query grouping and aggregations #3722

Closed
wants to merge 1 commit into from

Conversation

jmarkos
Copy link

@jmarkos jmarkos commented Sep 23, 2015

This PR depends on infinispan/protostream#24.

Several tests fail:
testAggregateDate
testComplexQuery
testComplexQuery2
testDateFilteringWithGroupBy
testEmbeddedAvg
testEmbeddedCount1
testEmbeddedCount2
testEmbeddedGlobalAvg
testEmbeddedGlobalCount
testEmbeddedGlobalMax
testEmbeddedGlobalSum
testEmbeddedMax
testEmbeddedSum

The first 4 fail because of https://issues.jboss.org/browse/ISPN-5787 , the rest is caused by https://issues.jboss.org/browse/ISPN-5682 (the issue is really about using aggregations on embedded entities).

@tristantarrant
Copy link
Member

This needs a protostream 3.0.4 from @anistor

@anistor
Copy link
Member

anistor commented Sep 25, 2015

protostream ready.

@jmarkos
Copy link
Author

jmarkos commented Sep 30, 2015

Rebased, bumped protostream version.

@danberindei
Copy link
Member

@jmarkos Still 200+ failures in CI

@jmarkos
Copy link
Author

jmarkos commented Oct 7, 2015

@danberindei They require https://issues.jboss.org/browse/ISPN-5682, so lets just wait I guess.

@anistor
Copy link
Member

anistor commented Jan 4, 2016

Closing and reopening here: #3919

I've managed to fix all tests except QueryDslConditionsTest.testComplexQuery2. Maybe @jmarkos can try it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants