Bugzilla – Attachment 132056 Details for
Bug 29507
Speed up auto renew cronjob via parallel processing
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 29507: Speed up auto renew cronjob via parallel processing
Bug-29507-Speed-up-auto-renew-cronjob-via-parallel.patch (text/plain), 15.04 KB, created by
Nick Clemens (kidclamp)
on 2022-03-23 15:17:48 UTC
(
hide
)
Description:
Bug 29507: Speed up auto renew cronjob via parallel processing
Filename:
MIME Type:
Creator:
Nick Clemens (kidclamp)
Created:
2022-03-23 15:17:48 UTC
Size:
15.04 KB
patch
obsolete
>From 99e74d75a8d2454b43a0f3e545b97fae29b9ea89 Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Wed, 23 Mar 2022 15:16:34 +0000 >Subject: [PATCH] Bug 29507: Speed up auto renew cronjob via parallel > processing > >The cron can take a very long time to run on systems with many issues. For example, a partner with ~250k auto_renew issues is taking about 9 hours to run. > >If we run that same number of holds in 5 parallel chunks ( splitting the number of issues as evenly as possible, it could take under 2 hours. > >Test Plan: >0) Install the Perl library Parallel::ForkManager >1) Generate a number of issues marked for auto_renew >2) Run the automatic_renewals.pl, use the `time` utility to track how much time it took to run >3) Set parallel_loops to 10 in auto_renew_cronjob section of config in koha-conf >4) Repeat step 2, note the improvement in speed >5) Experiment with other values >--- > debian/templates/koha-conf-site.xml.in | 4 + > etc/koha-conf.xml | 4 + > misc/cronjobs/automatic_renewals.pl | 219 +++++++++++++++---------- > 3 files changed, 141 insertions(+), 86 deletions(-) > >diff --git a/debian/templates/koha-conf-site.xml.in b/debian/templates/koha-conf-site.xml.in >index 6525e4ce97..11e9ec7528 100644 >--- a/debian/templates/koha-conf-site.xml.in >+++ b/debian/templates/koha-conf-site.xml.in >@@ -449,5 +449,9 @@ __END_SRU_PUBLICSERVER__ > <vhost>__MESSAGE_BROKER_VHOST__</vhost> > </message_broker> > >+ <auto_renew_cronjob> >+ <parallel_loops_count>1</parallel_loops_count> >+ </auto_renew_cronjob> >+ > </config> > </yazgfs> >diff --git a/etc/koha-conf.xml b/etc/koha-conf.xml >index 118b6b8207..c66cf90596 100644 >--- a/etc/koha-conf.xml >+++ b/etc/koha-conf.xml >@@ -266,5 +266,9 @@ > <vhost></vhost> > </message_broker> > >+ <auto_renew_cronjob> >+ <parallel_loops_count>1</parallel_loops_count> >+ </auto_renew_cronjob> >+ > </config> > </yazgfs> >diff --git a/misc/cronjobs/automatic_renewals.pl b/misc/cronjobs/automatic_renewals.pl >index c954aa9714..c8174eed94 100755 >--- a/misc/cronjobs/automatic_renewals.pl >+++ b/misc/cronjobs/automatic_renewals.pl >@@ -76,6 +76,7 @@ chosen 'Digests only' on the advance messages. > > use Modern::Perl; > use Pod::Usage qw( pod2usage ); >+use Parallel::ForkManager; > use Getopt::Long qw( GetOptions ); > > use Koha::Script -cron; >@@ -132,112 +133,158 @@ $verbose = 1 unless $verbose or $confirm; > print "Test run only\n" unless $confirm; > > print "getting auto renewals\n" if $verbose; >-my $auto_renews = Koha::Checkouts->search({ auto_renew => 1, 'patron.autorenew_checkouts' => 1 },{ join => 'patron'}); >-print "found " . $auto_renews->count . " auto renewals\n" if $verbose; >+my @auto_renews = Koha::Checkouts->search({ auto_renew => 1, 'patron.autorenew_checkouts' => 1 },{ join => 'patron', order_by => 'me.borrowernumber'})->as_list; >+print "found " . scalar @auto_renews . " auto renewals\n" if $verbose; > > my $renew_digest = {}; > my %report; >-while ( my $auto_renew = $auto_renews->next ) { >- print "examining item '" . $auto_renew->itemnumber . "' to auto renew\n" if $verbose; > >- my $borrower_preferences; >- $borrower_preferences = C4::Members::Messaging::GetMessagingPreferences( { borrowernumber => $auto_renew->borrowernumber, >- message_name => 'auto_renewals' } ) if $send_notices_pref eq 'preferences'; >- >- $send_notices = 1 if !$send_notices && $send_notices_pref eq 'preferences' && $borrower_preferences && $borrower_preferences->{transports} && $borrower_preferences->{transports}->{email}; >- >- # CanBookBeRenewed returns 'auto_renew' when the renewal should be done by this script >- my ( $ok, $error ) = CanBookBeRenewed( $auto_renew->borrowernumber, $auto_renew->itemnumber, undef, 1 ); >- if ( $error eq 'auto_renew' ) { >- if ($verbose) { >- say sprintf "Issue id: %s for borrower: %s and item: %s %s be renewed.", >- $auto_renew->issue_id, $auto_renew->borrowernumber, $auto_renew->itemnumber, $confirm ? 'will' : 'would'; >- } >- if ($confirm){ >- my $date_due = AddRenewal( $auto_renew->borrowernumber, $auto_renew->itemnumber, $auto_renew->branchcode, undef, undef, undef, 0 ); >- $auto_renew->auto_renew_error(undef)->store; >- } >- push @{ $report{ $auto_renew->borrowernumber } }, $auto_renew unless $send_notices_pref ne 'cron' && (!$borrower_preferences || !$borrower_preferences->{transports} || !$borrower_preferences->{transports}->{email} || $borrower_preferences->{'wants_digest'}); >- } elsif ( $error eq 'too_many' >- or $error eq 'on_reserve' >- or $error eq 'restriction' >- or $error eq 'overdue' >- or $error eq 'too_unseen' >- or $error eq 'auto_account_expired' >- or $error eq 'auto_too_late' >- or $error eq 'auto_too_much_oweing' >- or $error eq 'auto_too_soon' >- or $error eq 'item_denied_renewal' ) { >- if ( $verbose ) { >- say sprintf "Issue id: %s for borrower: %s and item: %s %s not be renewed. (%s)", >- $auto_renew->issue_id, $auto_renew->borrowernumber, $auto_renew->itemnumber, $confirm ? 'will' : 'would', $error; >- } >- if ( not $auto_renew->auto_renew_error or $error ne $auto_renew->auto_renew_error ) { >- $auto_renew->auto_renew_error($error)->store if $confirm; >- push @{ $report{ $auto_renew->borrowernumber } }, $auto_renew >- if $error ne 'auto_too_soon' && ($send_notices_pref eq 'cron' || ($borrower_preferences && $borrower_preferences->{transports} && $borrower_preferences->{transports}->{email} && !$borrower_preferences->{'wants_digest'})); # Do not notify if it's too soon >+my $cron_options = C4::Context->config('auto_renew_cronjob'); >+my $loops = $cron_options ? $cron_options->{parallel_loops_count} // 1 : 1; >+ >+# Split the list of issues into chunks to run in parallel >+my @chunks; >+if ( $loops > 1 ) { >+ my $i = 0; >+ my $borrowernumber = 0; >+ while (@auto_renews){ >+ my $auto_renew = pop(@auto_renews); >+ if( $borrowernumber != $auto_renew->borrowernumber ){ >+ $i++ if $borrowernumber; >+ $borrowernumber = $auto_renew->borrowernumber; > } >+ $i = 0 if $i >= $loops; >+ push( @{ $chunks[$i] }, $auto_renew ); > } > >- if ( $error ne 'auto_too_soon' && $borrower_preferences && $borrower_preferences->{transports} && $borrower_preferences->{transports}->{email} && $borrower_preferences->{'wants_digest'} ) { >- # cache this one to process after we've run through all of the items. >- if ($digest_per_branch) { >- $renew_digest->{ $auto_renew->branchcode }->{ $auto_renew->borrowernumber }->{success}++ if $error eq 'auto_renew'; >- $renew_digest->{ $auto_renew->branchcode }->{ $auto_renew->borrowernumber }->{error}++ unless $error eq 'auto_renew'; >- push @{$renew_digest->{ $auto_renew->branchcode }->{ $auto_renew->borrowernumber }->{issues}}, $auto_renew->itemnumber; >- } else { >- $renew_digest->{ $auto_renew->borrowernumber }->{success} ++ if $error eq 'auto_renew'; >- $renew_digest->{ $auto_renew->borrowernumber }->{error}++ unless $error eq 'auto_renew'; >- push @{$renew_digest->{ $auto_renew->borrowernumber }->{issues}}, $auto_renew->itemnumber; >- } >- } >+ my $pm = Parallel::ForkManager->new($loops); > >+ DATA_LOOP: >+ foreach my $chunk (@chunks) { >+ my $pid = $pm->start and next DATA_LOOP; >+ _ProcessRenewals( $chunk ); >+ $pm->finish; >+ } >+ $pm->wait_all_children; >+} else { >+ _ProcessRenewals( \@auto_renews ); > } > >-if ( $send_notices && $confirm ) { >- for my $borrowernumber ( keys %report ) { >- my $patron = Koha::Patrons->find($borrowernumber); >- for my $issue ( @{ $report{$borrowernumber} } ) { >- my $item = Koha::Items->find( $issue->itemnumber ); >- my $letter = C4::Letters::GetPreparedLetter( >- module => 'circulation', >- letter_code => 'AUTO_RENEWALS', >- tables => { >- borrowers => $patron->borrowernumber, >- issues => $issue->itemnumber, >- items => $issue->itemnumber, >- biblio => $item->biblionumber, >- }, >- lang => $patron->lang, >- ); >+=head1 METHODS > >- my $library = Koha::Libraries->find( $patron->branchcode ); >- my $admin_email_address = $library->from_email_address; >+=head2 _ProcessRenewals > >- C4::Letters::EnqueueLetter( >- { letter => $letter, >- borrowernumber => $borrowernumber, >- message_transport_type => 'email', >- from_address => $admin_email_address, >- } >- ); >+ Internal method to process the queue in chunks >+ >+=cut >+ >+sub _ProcessRenewals { >+ >+ my $auto_renew_issues = shift; >+ >+ foreach my $auto_renew (@$auto_renew_issues) { >+ print "examining item '" . $auto_renew->itemnumber . "' to auto renew\n" if $verbose; >+ >+ my $borrower_preferences; >+ $borrower_preferences = C4::Members::Messaging::GetMessagingPreferences( { borrowernumber => $auto_renew->borrowernumber, >+ message_name => 'auto_renewals' } ) if $send_notices_pref eq 'preferences'; >+ >+ $send_notices = 1 if !$send_notices && $send_notices_pref eq 'preferences' && $borrower_preferences && $borrower_preferences->{transports} && $borrower_preferences->{transports}->{email}; >+ >+ # CanBookBeRenewed returns 'auto_renew' when the renewal should be done by this script >+ my ( $ok, $error ) = CanBookBeRenewed( $auto_renew->borrowernumber, $auto_renew->itemnumber, undef, 1 ); >+ if ( $error eq 'auto_renew' ) { >+ if ($verbose) { >+ say sprintf "Issue id: %s for borrower: %s and item: %s %s be renewed.", >+ $auto_renew->issue_id, $auto_renew->borrowernumber, $auto_renew->itemnumber, $confirm ? 'will' : 'would'; >+ } >+ if ($confirm){ >+ my $date_due = AddRenewal( $auto_renew->borrowernumber, $auto_renew->itemnumber, $auto_renew->branchcode, undef, undef, undef, 0 ); >+ $auto_renew->auto_renew_error(undef)->store; >+ } >+ push @{ $report{ $auto_renew->borrowernumber } }, $auto_renew unless $send_notices_pref ne 'cron' && (!$borrower_preferences || !$borrower_preferences->{transports} || !$borrower_preferences->{transports}->{email} || $borrower_preferences->{'wants_digest'}); >+ } elsif ( $error eq 'too_many' >+ or $error eq 'on_reserve' >+ or $error eq 'restriction' >+ or $error eq 'overdue' >+ or $error eq 'too_unseen' >+ or $error eq 'auto_account_expired' >+ or $error eq 'auto_too_late' >+ or $error eq 'auto_too_much_oweing' >+ or $error eq 'auto_too_soon' >+ or $error eq 'item_denied_renewal' ) { >+ if ( $verbose ) { >+ say sprintf "Issue id: %s for borrower: %s and item: %s %s not be renewed. (%s)", >+ $auto_renew->issue_id, $auto_renew->borrowernumber, $auto_renew->itemnumber, $confirm ? 'will' : 'would', $error; >+ } >+ if ( not $auto_renew->auto_renew_error or $error ne $auto_renew->auto_renew_error ) { >+ $auto_renew->auto_renew_error($error)->store if $confirm; >+ push @{ $report{ $auto_renew->borrowernumber } }, $auto_renew >+ if $error ne 'auto_too_soon' && ($send_notices_pref eq 'cron' || ($borrower_preferences && $borrower_preferences->{transports} && $borrower_preferences->{transports}->{email} && !$borrower_preferences->{'wants_digest'})); # Do not notify if it's too soon >+ } >+ } >+ >+ if ( $error ne 'auto_too_soon' && $borrower_preferences && $borrower_preferences->{transports} && $borrower_preferences->{transports}->{email} && $borrower_preferences->{'wants_digest'} ) { >+ # cache this one to process after we've run through all of the items. >+ if ($digest_per_branch) { >+ $renew_digest->{ $auto_renew->branchcode }->{ $auto_renew->borrowernumber }->{success}++ if $error eq 'auto_renew'; >+ $renew_digest->{ $auto_renew->branchcode }->{ $auto_renew->borrowernumber }->{error}++ unless $error eq 'auto_renew'; >+ push @{$renew_digest->{ $auto_renew->branchcode }->{ $auto_renew->borrowernumber }->{issues}}, $auto_renew->itemnumber; >+ } else { >+ $renew_digest->{ $auto_renew->borrowernumber }->{success} ++ if $error eq 'auto_renew'; >+ $renew_digest->{ $auto_renew->borrowernumber }->{error}++ unless $error eq 'auto_renew'; >+ push @{$renew_digest->{ $auto_renew->borrowernumber }->{issues}}, $auto_renew->itemnumber; >+ } > } >+ > } > >- if ($digest_per_branch) { >- while (my ($branchcode, $digests) = each %$renew_digest) { >+ if ( $send_notices && $confirm ) { >+ for my $borrowernumber ( keys %report ) { >+ my $patron = Koha::Patrons->find($borrowernumber); >+ for my $issue ( @{ $report{$borrowernumber} } ) { >+ my $item = Koha::Items->find( $issue->itemnumber ); >+ my $letter = C4::Letters::GetPreparedLetter( >+ module => 'circulation', >+ letter_code => 'AUTO_RENEWALS', >+ tables => { >+ borrowers => $patron->borrowernumber, >+ issues => $issue->itemnumber, >+ items => $issue->itemnumber, >+ biblio => $item->biblionumber, >+ }, >+ lang => $patron->lang, >+ ); >+ >+ my $library = Koha::Libraries->find( $patron->branchcode ); >+ my $admin_email_address = $library->from_email_address; >+ >+ C4::Letters::EnqueueLetter( >+ { letter => $letter, >+ borrowernumber => $borrowernumber, >+ message_transport_type => 'email', >+ from_address => $admin_email_address, >+ } >+ ); >+ } >+ } >+ >+ if ($digest_per_branch) { >+ while (my ($branchcode, $digests) = each %$renew_digest) { >+ send_digests({ >+ digests => $digests, >+ branchcode => $branchcode, >+ letter_code => 'AUTO_RENEWALS_DGST', >+ }); >+ } >+ } else { > send_digests({ >- digests => $digests, >- branchcode => $branchcode, >+ digests => $renew_digest, > letter_code => 'AUTO_RENEWALS_DGST', > }); > } >- } else { >- send_digests({ >- digests => $renew_digest, >- letter_code => 'AUTO_RENEWALS_DGST', >- }); > } >+ > } > > =head1 METHODS >-- >2.30.2
<b>You cannot view the attachment while viewing its details because your browser does not support IFRAMEs. <a href="attachment.cgi?id=132056">View the attachment on a separate page</a>.</b>
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 29507
:
127743
|
127744
|
132056
|
150692
|
157772
|
161314
|
161315
|
161326
|
161327
|
165091
|
165092
|
168362
|
168363
|
168381
|
168382