[koha-commits] main Koha release repository branch 3.22.x updated. v3.22.07-89-g7db0861

Thu Jun 23 09:40:06 CEST 2016

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, 3.22.x has been updated
       via  7db08616f2eb44debd794ac8d7d42b05728caa50 (commit)
       via  337e8874600f757fe197062fd2967d27316b8cd1 (commit)
       via  ecf692819448aaa3e88b4caa6a9bdb6b1370d2e5 (commit)
       via  d64d83732efb32b1c473442abbd1d0284a7f298c (commit)
       via  9208f251198be56db5083d1ab022d5bde1cf59bd (commit)
       via  2430d359266ebba2b2b696d67f00a7a39606e2d3 (commit)
       via  a54774cbf2a5157b54722444c629d70f0f915af0 (commit)
       via  6599ae3ddc2340ef050046e3162e6ca48a2e7802 (commit)
      from  c4be9951a7c15a12ba4901c2632681cec8e68978 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 7db08616f2eb44debd794ac8d7d42b05728caa50
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Mon May 2 12:26:44 2016 +0200

    Bug 16412: Cache undef in L1 only

    No need to cache undef in L2.
    Prevent numerous warnings like:
    Use of uninitialized value in subroutine entry at /usr/share/koha/masterclone/Koha/Cache.pm line 284

    Test plan:
    Run t/Cache.t
    Run t/db_dependent/sysprefs.t

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Amended patch: remove trailing space char and replace 'you' with 'we' to
    match existing wordings.

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

    Signed-off-by: Jesse Weaver <jweaver at bywatersolutions.com>
    (cherry picked from commit 7a452ac4149809c212d861ee8b08c3931f23234a)
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

commit 337e8874600f757fe197062fd2967d27316b8cd1
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon May 2 12:39:29 2016 +0100

    Bug 16412: Make sure Koha::Cache->set_in_cache will not warn anymore

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

    Signed-off-by: Jesse Weaver <jweaver at bywatersolutions.com>
    (cherry picked from commit 53e410b14e2a3942bb577eff81c55799343d10d1)
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

commit ecf692819448aaa3e88b4caa6a9bdb6b1370d2e5
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Fri Apr 8 13:37:46 2016 +0100

    Bug 16221: follow-up for changes made by bug 16229

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    (cherry picked from commit 7f52638129bd48ab7f38dc809e7ece78773e8b91)
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

commit d64d83732efb32b1c473442abbd1d0284a7f298c
Author: Jacek Ablewicz <abl at biblos.pk.edu.pl>
Date:   Thu Apr 7 10:26:31 2016 +0200

    Bug 16221: Use Storable::dclone() instead of Clone::clone() for L1 cache deep-copying mode

    Storable dclone() is over 3x faster then Clone::clone() when used for
    creating a deep copies of the big, complex data structures like MARC
    frameworks. See also Bug 16044 comments #18 & #20.

    This is a trivial version of Bug 16166. Performance test
    results (see Bug 16140 comment #23 for test setup description):

       master
       146.29 (14.64+14.53+14.51+14.57+14.72+14.56+14.53+15.06+14.56+14.61)

       master + Bug 16221
       96.5 (9.77+9.63+9.77+9.77+9.68+9.05+9.68+9.83+9.63+9.69)

       master + Bug 16166
       90.7 (9.15+9.10+9.16+8.41+9.19+9.19+9.11+9.19+9.02+9.18)

    Bug 16166 is a bit faster and more comprehensive implementation, but
    also more complex and harder to test/evaluate.

    Test plan:

    1) apply patch
    2) profile GetMarcStructure() calls before / after patch, e.g. by
    running some script which calls it often (like catalogue search
    w/ XSLT processing turned on, etc.)

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    (cherry picked from commit 0c6f23a6ce45183c6396925f6018ec31090ed2cb)
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

commit 9208f251198be56db5083d1ab022d5bde1cf59bd
Author: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
Date:   Thu Apr 14 10:47:31 2016 -0300

    Bug 16229: (QA) POD fixes

    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>

    Signed-off-by: Brendan Gallagher <bredan at bywatersolutions.com>
    (cherry picked from commit 40ec26dc6cbc2ac2634ca0e8111476bbd6f62fc7)
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

commit 2430d359266ebba2b2b696d67f00a7a39606e2d3
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Fri Apr 8 12:54:44 2016 +0100

    Bug 16229: Add the unsafe flag to set_in_cache

    Could be useful later.

    Signed-off-by: Jacek Ablewicz <abl at biblos.pk.edu.pl>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>

    Signed-off-by: Brendan Gallagher <bredan at bywatersolutions.com>
    (cherry picked from commit 22c907230af9821d84127f9e2afbea9757aed2fa)
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

commit a54774cbf2a5157b54722444c629d70f0f915af0
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Fri Apr 8 12:43:48 2016 +0100

    Bug 16229: Deep copy on setting in cache

    Koha::Cache->set_in_cache should deep copy (if needed) to avoid the
    value which has been set in cache to be unintentionally modified later.

    Signed-off-by: Jacek Ablewicz <abl at biblos.pk.edu.pl>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>

    Signed-off-by: Brendan Gallagher <bredan at bywatersolutions.com>
    (cherry picked from commit 33ad907a820debc87bf2629cc1eed4a38b3b4905)
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

commit 6599ae3ddc2340ef050046e3162e6ca48a2e7802
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Fri Apr 8 13:08:55 2016 +0100

    Bug 16229: Deep copy on first L2 fetch

    When a value exists in L2 cache but not in L1 cache, it should be deep
    copied if needed (i.e. not a scalar). Otherwise the calling code is able
    to modify the value in cache.
    Note that is theoretical, it's possible that no code does that.

    Signed-off-by: Jacek Ablewicz <abl at biblos.pk.edu.pl>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>

    Signed-off-by: Brendan Gallagher <bredan at bywatersolutions.com>
    (cherry picked from commit 956dc953b52a1fd525bc75682bbdfe6848683ace)
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

-----------------------------------------------------------------------

Summary of changes:
 Koha/Cache.pm |   46 +++++++++++++++++++++++++++++++++-------------
 t/Cache.t     |   30 ++++++++++++++++++++++++++++--
 2 files changed, 61 insertions(+), 15 deletions(-)

hooks/post-receive
-- 
main Koha release repository