Skip to content

#371 - Add DatabaseOperation #372

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
Merged
merged 36 commits into from
Apr 18, 2022
Merged

#371 - Add DatabaseOperation #372

merged 36 commits into from
Apr 18, 2022

Conversation

@garciparedes
Copy link
Member

@garciparedes garciparedes commented Apr 6, 2022

No description provided.

Sergio García Prado added 2 commits April 6, 2022 09:32
ISSUE #371
e8df1cb 
* Add `DatabaseOperation`.
* Add `AiopgDatabaseOperation`.
* Add `ComposedDatabaseOperation`.
* Add `EventRepositoryOperationFactory`.
* Add `AiopgEventRepositoryOperationFactory`.
* Refactor `PostgreSqlEventRepository` as `DatabaseEventRepository`.
Merge remote-tracking branch 'origin/issue-367-add-database-client' i…
7b3ac81 
…nto issue-371-add-database-operation

# Conflicts:
#	packages/core/minos-microservice-common/minos/common/database/clients/abc.py
#	packages/core/minos-microservice-common/minos/common/database/clients/aiopg.py
@garciparedes garciparedes added this to the v0.7.x milestone Apr 6, 2022
@garciparedes garciparedes self-assigned this Apr 6, 2022
@garciparedes garciparedes linked an issue Apr 6, 2022 that may be closed by this pull request
Add DatabaseOperation class #371
Closed
@garciparedes garciparedes marked this pull request as draft April 6, 2022 07:40
Sergio García Prado added 6 commits April 6, 2022 10:09
ISSUE #371
02036b7 
* Fix broken tests.
ISSUE #371
d624b0c 
* Refactor `PostgreSqlTransactionRepository` as `DatabaseTransactionRepository`.
* Add `TransactionRepositoryOperationFactory`.
* Add `AiopgTransactionRepositoryOperationFactory`.
Merge remote-tracking branch 'origin/issue-367-add-database-client' i…
a07ebdf 
…nto issue-371-add-database-operation
ISSUE #371
13672b3 
* Fix bug related with a query.
ISSUE #371
d701d17 
* Refactor `PostgreSqlBrokerSubscriberDuplicateValidator`.
Merge remote-tracking branch 'origin/issue-367-add-database-client' i…
784f518 
…nto issue-371-add-database-operation
@restyled-io restyled-io bot mentioned this pull request Apr 6, 2022
Closed
Sergio García Prado added 16 commits April 6, 2022 13:08
ISSUE #371
2a10dfd 
* Replace Postgres' listen/notify by `asyncio.Event`.
ISSUE #371
1e2f90f 
* Refactor `PostgreSqlBrokerQueue`.
ISSUE #371
a4dcfb1 
* Minor change.
ISSUE #371
d4a6b69 
* Minor improvements.
ISSUE #371
340e668 
* Add `AiopgSnapshotRepositoryOperationFactory`.
ISSUE #371
0001cf9 
* Integrate `AiopgSnapshotRepositoryOperationFactory` and `PostgreSqlSnapshotQueryBuilder`.
ISSUE #371
90499e3 
* Rename classes.
ISSUE #371
7daa667 
* Rename classes (2).
ISSUE #371
6ef3f81 
* Rename classes (3).
ISSUE #371
9a50a56 
* Add base classes.
* Rename classes (4).
ISSUE #371
e5fa8dc 
* Improve `DatabaseLock`.
* Refactor `DatabaseClient.execute` method.
ISSUE #371
c3499b5 
* Rename `query_factory` as `operation_factory`.
ISSUE #371
3fa7f94 
* Minor improvements.
ISSUE #371
c4fcac3 
* Minor improvements (2).
ISSUE #371
afb8bba 
* Add `DatabaseOperationFactory` base class.
* Add `register_factory` adn `get_factory` methods to `DatabaseClient`.
ISSUE #371
2a9c383 
* Now `timeout` and `lock` is managed by the `DatabaseClient` instead of by the `AiopgDatabaseClient`.
Sergio García Prado added 10 commits April 7, 2022 09:52
ISSUE #371
7268b2d 
* Fix minor bugs.
ISSUE #371
4bf4b4c 
* Add `ManageDatabaseOperationFactory`.
ISSUE #371
ed522a5 
* Fix bugs.
ISSUE #371
5ec3822 
* Remove `PostgreSqlEventRepository`.
* Remove `PostgreSqlSnapshotReader`.
* Remove `PostgreSqlSnapshotRepository`.
* Remove `PostgreSqlSnapshotQueryBuilder`.
* Remove `PostgreSqlSnapshotWriter`.
* Remove `PostgreSqlSnapshotSetup`.
* Remove `PostgreSqlTransactionRepository`.
ISSUE #371
665a424 
* Rename `PostgresAsyncTestCase` as `DatabaseMinosTestCase`
ISSUE #371
e1d3bf9 
* Remove `PostgreSql*` like classes.
ISSUE #371
83bdc06 
* Allow to setup `DatabaseOperationFactory` at `DatabaseMixin` from directly generics.
ISSUE #371
53ad4c5 
* Add missing docstring.
ISSUE #371
285dbc0 
* Increase coverage.
ISSUE #371
b831356 
* Increase coverage (2).
@garciparedes garciparedes requested a review from a team April 8, 2022 07:02
@garciparedes garciparedes marked this pull request as ready for review April 8, 2022 07:02
Base automatically changed from issue-367-add-database-client to 0.7.0 April 18, 2022 09:10
@garciparedes garciparedes merged commit e6e931e into 0.7.0 Apr 18, 2022
@garciparedes garciparedes deleted the issue-371-add-database-operation branch April 18, 2022 09:10
@garciparedes garciparedes removed the request for review from a team April 18, 2022 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@garciparedes garciparedes

Labels

databases minos-microservice-common

Projects

None yet

Milestone

v0.7.x

Development

Successfully merging this pull request may close these issues.

Add DatabaseOperation class

2 participants

@garciparedes