Reviewed & tested by the community
Project:
Igbinary
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
15 Dec 2016 at 13:57 UTC
Updated:
19 May 2025 at 13:16 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
mkalkbrennerComment #3
andypostWhy you need such hack instead of changing serializer at PHP level? It looks like tremendous overhead for build-in operation
Comment #4
fabianx commentedI think we should do that in core:
#3014514: Make igbinary the default serializer if available, it saves 50% time on unserialize and memory footprint
Comment #5
socialnicheguru commentedPer @andypost I do not need this patch if I update my services.yml to use the proper serializer. Is my interpretation correct?
Comment #6
mkalkbrennerNo. You can do everything in your site specific services.yml.
This patch was just a convenience function.
Comment #7
andypostAs core 11.0/10.3 has serializer configurable #839444: Make serializer customizable for Cache\DatabaseBackend
maybe it needs follow-up to fix arguments
Comment #8
grimreaperHello,
As the 8.x-1.x branch is no more supported and as the 2.0.x module has a different approach, is this issue still relevant? Could it be closed?
Comment #9
socialnicheguru commentedD10.3/11 added #839444: Make serializer customizable for Cache\DatabaseBackend.
Does it have any impact on this module version 2.x?
Comment #10
andypostI think we need to add PECL extension to CI images to make ball roll
Comment #11
andypostnow all 8.3-8.5 images has igbinary PECL enabled