Attachment 33314 Details for Bug 13214 – Bug 13214 - Improve performance of acqui/addorderiso2709.pl, stop fetching all import_records needlessly

[patch] Bug 13214 - Improve performance of acqui/addorderiso2709.pl, stop fetching all import_records needlessly

Bug-13214---Improve-performance-of-acquiaddorderis.patch (text/plain), 1.48 KB, created by Olli-Antti Kivilahti on 2014-11-06 14:48:41 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Olli-Antti Kivilahti

Created: 2014-11-06 14:48:41 UTC

Size: 1.48 KB

patch

obsolete

>From c537ba575636802e8e207f8dcc665dafd1fa15ea Mon Sep 17 00:00:00 2001
>From: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi>
>Date: Thu, 6 Nov 2014 16:38:37 +0200
>Subject: [PATCH] Bug 13214 - Improve performance of acqui/addorderiso2709.pl,
> stop fetching all import_records needlessly
>
>TEST PLAN:
>
>1. Load 167 staged files to Koha.
>2. Try to make an acquisiton from a staged file.
>3. Wait 16s for the acqui/addorderiso2709.pl -view to load.
>
>AFTER PATCH:
>
>1. Load 167 staged files to Koha.
>2. Try to make an acquisiton from a staged file.
>3. Wait 1.6s for the acqui/addorderiso2709.pl -view to load.
>---
> acqui/addorderiso2709.pl |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/acqui/addorderiso2709.pl b/acqui/addorderiso2709.pl
>index 1a96e6c..3dd39ac 100755
>--- a/acqui/addorderiso2709.pl
>+++ b/acqui/addorderiso2709.pl
>@@ -339,7 +339,7 @@ sub import_batches_list {
>     foreach my $batch (@$batches) {
>         if ( $batch->{'import_status'} =~ /^staged$|^reverted$/ ) {
>             # check if there is at least 1 line still staged
>-            my $stagedList=GetImportRecordsRange($batch->{'import_batch_id'}, undef, undef, $batch->{import_status}, { order_by_direction => 'ASC' });
>+            my $stagedList=GetImportRecordsRange($batch->{'import_batch_id'}, undef, 1, $batch->{import_status}, { order_by_direction => 'ASC' });
>             if (scalar @$stagedList) {
>                 push @list, {
>                         import_batch_id => $batch->{'import_batch_id'},
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 13214: 33314 | 33322 | 33794