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

Allow read from postgres replica if there's #470

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ganesanarun
Copy link
Contributor

@ganesanarun ganesanarun commented Aug 4, 2020

It's configurable in case if we have replica then we could use to read from replica, otherwise by default it will use the master configuration from read postgres client too.

Right now, it's being enabled only for ConsentRequestRepository. We can do the same thing for other repositories once we tested out.

@ganesanarun ganesanarun requested a review from a team as a code owner August 4, 2020 10:16
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 4, 2020

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 1 Code Smell

44.4% 44.4% Coverage
0.0% 0.0% Duplication

It's configurable in case if we have replica
then we could use to read from replica, otherwise
by default it will use the master configuration from
read postgres client too
@mddubey mddubey force-pushed the read-from-replica-db branch from 147a139 to f10cd31 Compare August 21, 2020 09:29
Copy link
Contributor Author

Codacy Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- consent/src/main/java/in/projecteka/consentmanager/properties/DbOptions.java  2
         

Clones added
============
- consent/src/main/java/in/projecteka/consentmanager/user/OtpAttemptService.java  1
         

See the complete overview on Codacy

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

Successfully merging this pull request may close these issues.

2 participants