-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: filter with datasetId parameter #1177
feat: filter with datasetId parameter #1177
Conversation
Test Results 57 files ± 0 57 suites ±0 1m 21s ⏱️ +3s Results for commit e0602bc. ± Comparison against base commit 5c63a14. This pull request removes 202 and adds 62 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
search-service/src/test/kotlin/com/egm/stellio/search/web/EntityHandlerTests.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/model/EntitiesQuery.kt
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/model/Query.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/util/EntitiesQueryUtils.kt
Outdated
Show resolved
Hide resolved
shared/src/main/kotlin/com/egm/stellio/shared/model/ExpandedEntity.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/util/EntitiesQueryUtils.kt
Outdated
Show resolved
Hide resolved
shared/src/main/kotlin/com/egm/stellio/shared/model/ExpandedEntity.kt
Outdated
Show resolved
Hide resolved
shared/src/main/kotlin/com/egm/stellio/shared/model/ExpandedEntity.kt
Outdated
Show resolved
Hide resolved
shared/src/main/kotlin/com/egm/stellio/shared/model/ExpandedEntity.kt
Outdated
Show resolved
Hide resolved
shared/src/main/kotlin/com/egm/stellio/shared/model/ExpandedEntity.kt
Outdated
Show resolved
Hide resolved
shared/src/main/kotlin/com/egm/stellio/shared/model/ExpandedEntity.kt
Outdated
Show resolved
Hide resolved
shared/src/main/kotlin/com/egm/stellio/shared/model/ExpandedEntity.kt
Outdated
Show resolved
Hide resolved
subscription-service/src/main/kotlin/com/egm/stellio/subscription/model/Subscription.kt
Outdated
Show resolved
Hide resolved
|
worth having a look at #1177 (comment) |
it will start from here: https://github.com/stellio-hub/stellio-context-broker/blob/develop/search-service/src/main/kotlin/com/egm/stellio/search/service/QueryService.kt#L56 and https://github.com/stellio-hub/stellio-context-broker/blob/develop/search-service/src/main/kotlin/com/egm/stellio/search/service/QueryService.kt#L124 Then, of course, SQL requests will have to be enhanced to support |
search-service/src/main/kotlin/com/egm/stellio/search/service/EntityPayloadService.kt
Outdated
Show resolved
Hide resolved
...ervice/src/test/kotlin/com/egm/stellio/search/service/TemporalEntityAttributeServiceTests.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/service/TemporalEntityAttributeService.kt
Outdated
Show resolved
Hide resolved
@ranim-n I added some chore and fix commits, I let you have a look. |
Quality Gate passedIssues Measures |
I can't approve because I am the author of the PR but LGTM |
…itiesQueryUtils.kt Co-authored-by: Benoit Orihuela <[email protected]>
…d attributes instances
…TemporalEntityAttributeServiceTests.kt Co-authored-by: Benoit Orihuela <[email protected]>
e0602bc
to
d6331ee
Compare
No description provided.