[Koha-bugs] [Bug 16715] Koha::Cache - Use Sereal for serialization

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16715

--- Comment #26 from Jacek Ablewicz <abl at biblos.pk.edu.pl> ---
(In reply to Jonathan Druart from comment #23)

>
> I have quickly tried this
> 
> diff --git a/Koha/Cache.pm b/Koha/Cache.pm
> index 0f85ed1..fa4f136 100644
> --- a/Koha/Cache.pm
> +++ b/Koha/Cache.pm
> @@ -40,7 +40,8 @@ use warnings;
>  use Carp;
>  use Module::Load::Conditional qw(can_load);
>  use Koha::Cache::Object;
> -use Sereal;
> +use Sereal::Encoder;
> +use Sereal::Decoder;
>  
>  use base qw(Class::Accessor);
>  
> 
> And it seems to work correctly.
> I can provide a patch if you need it.

That should work just fine, I used that variant for speed differences
comparisons vs Storable (I think it was on jessie, not entirely sure) without
encountering any problems.

Speaking of dependencies: what about some other files (debian/control*,
install_misc/*.packages) in the source tree which are referencing required
packages? I don't know if they may need some tweaking as well or not - or maybe
it's only being done en masse before releasing next packaged version?

-- 
You are receiving this mail because:
You are watching all bug changes.