taskain7 commented on code in PR #3027:
URL: https://github.com/apache/fineract/pull/3027#discussion_r1130875304
##########
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanRepository.java:
##########
@@ -84,11 +84,13 @@ public interface LoanRepository extends JpaRepository<Loan,
Long>, JpaSpecificat
// should follow the logic of
`FIND_ALL_NON_CLOSED_LOANS_BY_LAST_CLOSED_BUSINESS_DATE` query
String FIND_OLDEST_COB_PROCESSED_LOAN = "select loan.id,
loan.lastClosedBusinessDate from Loan loan where loan.loanStatus in
(100,200,300,303,304) and loan.lastClosedBusinessDate = (select
min(l.lastClosedBusinessDate) from Loan l where l"
- + ".loanStatus in (100,200,300,303,304) and
l.lastClosedBusinessDate <> :cobBusinessDate)";
+ + ".loanStatus in (100,200,300,303,304) and
l.lastClosedBusinessDate < :cobBusinessDate)";
- String FIND_ALL_NON_CLOSED_LOANS_BEHIND_BY_LOAN_IDS = "select loan.id,
loan.lastClosedBusinessDate from Loan loan where loan.id IN :loanIds and
loan.loanStatus in (100,200,300,303,304) and (loan.lastClosedBusinessDate <>
:cobBusinessDate or "
+ String FIND_ALL_NON_CLOSED_LOANS_BEHIND_OR_NULL_BY_LOAN_IDS = "select
loan.id, loan.lastClosedBusinessDate from Loan loan where loan.id IN :loanIds
and loan.loanStatus in (100,200,300,303,304) and (loan.lastClosedBusinessDate <
:cobBusinessDate or "
+ "loan.lastClosedBusinessDate is null)";
+ String FIND_ALL_NON_CLOSED_LOANS_BEHIND_BY_LOAN_IDS = "select loan.id,
loan.lastClosedBusinessDate from Loan loan where loan.id IN :loanIds and
loan.loanStatus in (100,200,300,303,304) and loan.lastClosedBusinessDate <
:cobBusinessDate";
Review Comment:
same as above
##########
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanRepository.java:
##########
@@ -84,11 +84,13 @@ public interface LoanRepository extends JpaRepository<Loan,
Long>, JpaSpecificat
// should follow the logic of
`FIND_ALL_NON_CLOSED_LOANS_BY_LAST_CLOSED_BUSINESS_DATE` query
String FIND_OLDEST_COB_PROCESSED_LOAN = "select loan.id,
loan.lastClosedBusinessDate from Loan loan where loan.loanStatus in
(100,200,300,303,304) and loan.lastClosedBusinessDate = (select
min(l.lastClosedBusinessDate) from Loan l where l"
- + ".loanStatus in (100,200,300,303,304) and
l.lastClosedBusinessDate <> :cobBusinessDate)";
+ + ".loanStatus in (100,200,300,303,304) and
l.lastClosedBusinessDate < :cobBusinessDate)";
- String FIND_ALL_NON_CLOSED_LOANS_BEHIND_BY_LOAN_IDS = "select loan.id,
loan.lastClosedBusinessDate from Loan loan where loan.id IN :loanIds and
loan.loanStatus in (100,200,300,303,304) and (loan.lastClosedBusinessDate <>
:cobBusinessDate or "
+ String FIND_ALL_NON_CLOSED_LOANS_BEHIND_OR_NULL_BY_LOAN_IDS = "select
loan.id, loan.lastClosedBusinessDate from Loan loan where loan.id IN :loanIds
and loan.loanStatus in (100,200,300,303,304) and (loan.lastClosedBusinessDate <
:cobBusinessDate or "
Review Comment:
will this work with more than 65K loan ids in the parameter on PostgreSQL?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
