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
[FINERACT-1678] Cob job skeleton #2491
Conversation
ec1faa0
to
3fe023d
Compare
408206e
to
f2c879d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
f83acdc
to
6def2cb
Compare
@taskain7 Would you check why the jobs are timing out? |
I'm on it |
fb85cb8
to
b465684
Compare
fineract-provider/src/main/java/org/apache/fineract/infrastructure/jobs/service/JobName.java
Outdated
Show resolved
Hide resolved
fineract-provider/src/main/java/org/apache/fineract/cob/loan/LoanCOBWorkerConfiguration.java
Outdated
Show resolved
Hide resolved
fineract-provider/src/main/java/org/apache/fineract/cob/loan/LoanCOBWorkerConfiguration.java
Outdated
Show resolved
Hide resolved
List<Integer> remainingData = new ArrayList<>(data); | ||
return () -> { | ||
if (remainingData.size() > 0) { | ||
return loanRepository.findById(remainingData.remove(0).longValue()).orElse(null); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to be an overkill to issue a single statement for every single ID, don't you think?
We shall rather grab a chunk size of Loans at once.
fineract-provider/src/main/java/org/apache/fineract/cob/loan/LoanCOBPartitioner.java
Show resolved
Hide resolved
...ract-provider/src/main/java/org/apache/fineract/cob/loan/LoanCOBInputChannelTransformer.java
Outdated
Show resolved
Hide resolved
fineract-provider/src/main/java/org/apache/fineract/cob/loan/LoanCOBMessage.java
Outdated
Show resolved
Hide resolved
...act-provider/src/main/java/org/apache/fineract/cob/loan/LoanCOBOutputChannelTransformer.java
Outdated
Show resolved
Hide resolved
...-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanRepository.java
Outdated
Show resolved
Hide resolved
fineract-provider/src/main/resources/db/changelog/tenant/parts/0037_add_loan_cob_job_data.xml
Outdated
Show resolved
Hide resolved
b465684
to
d8da59f
Compare
Description
This is a skeleton for the remote partitioned Loan COB Spring Batch job.