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

Union and plus operations should return a query of the common base type #49

Closed
sedovalx opened this issue Jul 4, 2018 · 1 comment
Closed

Union and plus operations should return a query of the common base type #49

sedovalx opened this issue Jul 4, 2018 · 1 comment
Assignees
@sedovalx

Comments

@sedovalx
Copy link
Contributor

sedovalx commented Jul 4, 2018

Otherwise, sorting over the result of the union/plus returns incomplete iterable. Such a sort returns items of the left part of the union/plus. It happens because the result of the union/plus inherits the entityType of the left part. When the result is passed into the queryEngine to sort over a property the engine returns only the entities of the passed entity type.
@sedovalx sedovalx self-assigned this Jul 4, 2018
sedovalx added a commit to sedovalx/xodus-dnq that referenced this issue Jul 4, 2018
@sedovalx
JetBrains#49 Renamed assertThat -> assertQuery
54b3eb8
sedovalx added a commit to sedovalx/xodus-dnq that referenced this issue Jul 4, 2018
@sedovalx
JetBrains#49 Tests of the union and concat
be6552b
sedovalx added a commit to sedovalx/xodus-dnq that referenced this issue Jul 4, 2018
@sedovalx
JetBrains#49 Getting a common ancestor of a pair of entity types
577b6c6
sedovalx added a commit to sedovalx/xodus-dnq that referenced this issue Jul 4, 2018
@sedovalx
JetBrains#49 Caching of common ancestors
ca9cc05
sedovalx added a commit to sedovalx/xodus-dnq that referenced this issue Jul 7, 2018
@sedovalx
JetBrains#49 Removed caching of common ancestors as seems it has no m…
5eeaf00 
…uch sense
@sedovalx
Copy link
Contributor Author

Merged in #50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sedovalx sedovalx

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sedovalx