Skip to content

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.

3 participants