Skip to content

Allow access to mongoDatabaseFactory used in ReactiveMongoTemplate #3540

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

Closed
christophstrobl opened this issue Jan 21, 2021 · 0 comments
Closed
Assignees
Labels
type: enhancement A general enhancement

Comments

@christophstrobl
Copy link
Member

By offering a getter method for the ReactiveMongoDatabaseFactory users subclassing ReactiveMongoTemplate could evaluate the current transaction state via ReactiveMongoDatabaseUtils.isTransactionActive(getDatabaseFactory()).

This change also aligns the reactive with the imperative template implementation.

@christophstrobl christophstrobl added the type: enhancement A general enhancement label Jan 21, 2021
@christophstrobl christophstrobl self-assigned this Jan 21, 2021
christophstrobl added a commit that referenced this issue Jan 21, 2021
By offering a getter method for the ReactiveMongoDatabaseFactory users subclassing ReactiveMongoTemplate could evaluate the current transaction state via ReactiveMongoDatabaseUtils.isTransactionActive(getDatabaseFactory()).
This change also aligns the reactive and imperative template implementation in that regard.

Closes #3540
@christophstrobl christophstrobl linked a pull request Jan 21, 2021 that will close this issue
@mp911de mp911de added this to the 3.1.4 (2020.0.4) milestone Feb 2, 2021
@mp911de mp911de closed this as completed in a33aece Feb 2, 2021
mp911de pushed a commit that referenced this issue Feb 2, 2021
By offering a getter method for the ReactiveMongoDatabaseFactory users subclassing ReactiveMongoTemplate could evaluate the current transaction state via ReactiveMongoDatabaseUtils.isTransactionActive(getDatabaseFactory()).
This change also aligns the reactive and imperative template implementation in that regard.

Closes #3540
Original pull request: #3541.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants