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
refactor(triplestore): ZIO-fying triplestore service (DSP-904) #2059
Merged
subotic
merged 87 commits into
main
from
wip/DEV-904-dsp-api-zio-fying-triplestore-service
Jul 1, 2022
Merged
Changes from 75 commits
Commits
Show all changes
87 commits
Select commit
Hold shift + click to select a range
c023754
add triplestore service interface
subotic e723107
refactor runtime initialisation
subotic 283c733
Revert "refactor runtime initialisation"
subotic 5990f04
test: raise timeout
subotic 8de966c
test: raise timeout
subotic 2d6308c
start refactoring of the triplestore service
subotic 3a3c38a
ongoing refactoring of the triplestore service
subotic 0648eab
ongoing refactoring of the triplestore manager
subotic 311d7b7
ongoing refactoring of repository updater
subotic fb4c04c
Merge branch 'main' into wip/DEV-904-dsp-api-zio-fying-triplestore-se…
subotic b7828ad
rename cache package
subotic 411310d
ongoing refactoring of the triplestore manager
subotic c0fb299
ongoing removing feature factory stuff
subotic 37985ab
ongoing removing feature factory stuff
subotic fb2bed8
ongoing feature factory
subotic 34c925f
ongoing remove feature factories
subotic 00b1f8b
remove feature factory
subotic 2e2d4f3
remove feature factory
subotic da0590f
fix config
subotic d0e03e0
test: triplestore connector (ongoing)
subotic 0e7d1bc
cleanup
subotic bbf4615
ongoing fixing AllData route
subotic 81cc799
add ADR
subotic 2a92fff
ongoing fixing AllData route
subotic b9a046a
ongoing fixing AllData route
subotic 186215c
fix AllData route
subotic bc9a577
remove test
subotic 4d45550
fix tests
subotic f5ad3b9
fix tests
subotic 3ca0672
fix tests
subotic a539a89
fix tests
subotic bbed179
add example specs
subotic 9bc62aa
fix tests
subotic 61232cf
Merge branch 'main' into wip/DEV-904-dsp-api-zio-fying-triplestore-se…
subotic 23bb0b1
remove feature factory stuff
subotic dc63361
try snapshot version of ZIO (ongoing)
subotic e77e5bc
clean up testcontainer usage
subotic 85d76dc
fix formatting
subotic 7c8d3e4
Merge branch 'main' into wip/DEV-904-dsp-api-zio-fying-triplestore-se…
subotic ff6eabf
Merge branch 'main' into wip/DEV-904-dsp-api-zio-fying-triplestore-se…
subotic 0532e9f
fix after merging of main
subotic c9756aa
Merge branch 'main' into wip/DEV-904-dsp-api-zio-fying-triplestore-se…
subotic f3b7afa
fix after merge (ongoing)
subotic a7398f7
fixes after merging (ongoing)
subotic ffe1f1a
fixes after merging
subotic 0157436
fix releasing of testcontainers
subotic 9fe6f54
fix: tests
subotic 54bede6
refactor test client service
subotic 923a500
fix more tests
subotic 4fc4e34
fix more tests
subotic e9aa4b3
Merge branch 'main' into wip/DEV-904-dsp-api-zio-fying-triplestore-se…
subotic 6aff30c
fix: app initialization
subotic 1dcfb53
apply fmt
subotic d327a9b
fix: only use logging in app actor
subotic 0ade0e0
fix: double logging
subotic 855b772
fix test initialization
subotic a0627b7
cleanup
subotic 0ba692a
change logging format
subotic 37cdd24
Merge branch 'main' into wip/DEV-904-dsp-api-zio-fying-triplestore-se…
subotic c85ec6c
fix merge conflicts
subotic 29bf2c9
bump ZIO version to 2.0
subotic 5ac4705
fixing ZIO 2.0 changes (ongoing)
subotic ff80343
fixing ZIO 2.0 changes (ongoing)
subotic 91256f8
fixing ZIO 2.0 changes (ongoing)
subotic 521b442
fixing ZIO 2.0 changes (ongoing)
subotic e4a3d09
fix sipi integration test
subotic 399f3b2
cleanup logging
subotic 1965010
bump zio versions
subotic ab36995
bump zio versions
subotic 5a5da2f
fix formatting
subotic ed20b47
Merge branch 'main' into wip/DEV-904-dsp-api-zio-fying-triplestore-se…
subotic f10c2f2
fix logging
subotic dc2f11d
trying to fix failing test
subotic 4ad4bda
bump scalatest
subotic c0eedd1
trying to fix testing
subotic 8982236
Update webapi/src/main/scala/org/knora/webapi/http/handler/KnoraExcep…
subotic cb9d3c0
Update webapi/src/main/scala/org/knora/webapi/http/handler/KnoraExcep…
subotic 3c974ab
delete tests
subotic 57ecc38
Merge branch 'wip/DEV-904-dsp-api-zio-fying-triplestore-service' of g…
subotic 335ebc7
cleanup
subotic 70abec9
trying to fix running of tests
subotic cdfea8b
trying to fix running of tests
subotic c62172a
trying to fix tests
subotic c123efd
Merge branch 'main' into wip/DEV-904-dsp-api-zio-fying-triplestore-se…
subotic 7d08b9d
test cleanup
subotic 25324fa
trying to fix tests
subotic f90ea35
cleanup
subotic File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
20 changes: 20 additions & 0 deletions
20
.../0004-change-triplestore-service-manager-and-fuseki-implementation-to-zlayer.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
# 4. Change Triplestore Service Manager and Fuseki implementation to ZLayer | ||
|
||
Date: 2022-05-23 | ||
|
||
## Status | ||
|
||
Accepted | ||
|
||
## Context | ||
|
||
Both `org.knora.webapi.store.triplestore.TriplestoreServiceManager` and `org.knora.webapi.store.triplestore.impl.TriplestoreServiceHttpConnectorImpl` | ||
where implemented as Akka-Actors. | ||
|
||
## Decision | ||
|
||
As part of the move from `Akka` to `ZIO`, it was decided that the `TriplestoreServiceManager` and the `TriplestoreServiceHttpConnectorImpl` is refactored using ZIO. | ||
|
||
## Consequences | ||
|
||
The usage from other actors stays the same. The actor messages and responses don't change. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Setting the case class to
final
here makes sense. Shall we do it for all Error case classes?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For those that we don't want to have any subclasses, we definitely should. Btw. This error is one of those that should be moved out of the shared project and to the triplestore service, as it is specific to it.