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

Clean up/consolidate Database Logic #16

Open
to11mtm opened this issue Jan 31, 2021 · 0 comments
Open

Clean up/consolidate Database Logic #16

to11mtm opened this issue Jan 31, 2021 · 0 comments

Comments

@to11mtm
Copy link
Member

to11mtm commented Jan 31, 2021

Because Linq2Db does not have the reactive stream capabilities that SlickDB has for JDBC side, We had to write some of our data access patterns differently. In the interests of getting a working product out we did not in all cases go with a 'consolidated' design where database access patterns are held in their own file, instead right now the various Journals and SnapshotStores are using Linq2Db directly.

It would not be a bad idea to consolidate this database logic into another class. Linq2Db thankfully has a lot of capabilities around syntactic sugar that make it easy to write composable functor-like classes.

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

No branches or pull requests

1 participant